From b9fc023ae7a24f25724381cec5f5e6cc71f68dd1 Mon Sep 17 00:00:00 2001
From: Chrison Simtian <chris.simon@myfoodbag.co.nz>
Date: Sat, 15 Aug 2026 11:14:38 +1200
Subject: [PATCH] Rename the Targets view to Build, under a versioned view id
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Groundwork for the Deployment and Run Configuration views: the container is
about to hold several views, so the single "Targets" view becomes "Build" and
moves to a namespaced id.

`falloutTargets` -> `fallout.build` would be a breaking change against a
released extension — it invalidates keybindings with a `view ==` when-clause,
saved layout state, and user-written when-contexts. Nothing has been published
yet, so there is no such state anywhere, which is exactly why the naming gets
settled now rather than after the first release.

The view's contents and behaviour are unchanged.

Closes #9

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
---
 package.json     | 16 ++++++++--------
 src/extension.ts |  2 +-
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/package.json b/package.json
index 39f5f4f..2dc82db 100644
--- a/package.json
+++ b/package.json
@@ -30,7 +30,7 @@
     "homepage": "https://fallout.build",
     "main": "./out/extension.js",
     "activationEvents": [
-        "onView:falloutTargets",
+        "onView:fallout.build",
         "workspaceContains:**/.fallout/temp/build-graph.json",
         "workspaceContains:**/.nuke/temp/build-graph.json"
     ],
@@ -47,8 +47,8 @@
         "views": {
             "fallout": [
                 {
-                    "id": "falloutTargets",
-                    "name": "Targets",
+                    "id": "fallout.build",
+                    "name": "Build",
                     "icon": "media/fallout.svg",
                     "contextualTitle": "Fallout"
                 }
@@ -56,7 +56,7 @@
         },
         "viewsWelcome": [
             {
-                "view": "falloutTargets",
+                "view": "fallout.build",
                 "contents": "No Fallout build graph found in this workspace.\nRun the build once to generate `build-graph.json`, then refresh.\n[Plan build (dry run)](command:fallout.planTarget)"
             }
         ],
@@ -96,24 +96,24 @@
             "view/title": [
                 {
                     "command": "fallout.showGraph",
-                    "when": "view == falloutTargets",
+                    "when": "view == fallout.build",
                     "group": "navigation@1"
                 },
                 {
                     "command": "fallout.refreshTargets",
-                    "when": "view == falloutTargets",
+                    "when": "view == fallout.build",
                     "group": "navigation@2"
                 }
             ],
             "view/item/context": [
                 {
                     "command": "fallout.runTarget",
-                    "when": "view == falloutTargets && viewItem == falloutTarget",
+                    "when": "view == fallout.build && viewItem == falloutTarget",
                     "group": "inline@1"
                 },
                 {
                     "command": "fallout.goToTarget",
-                    "when": "view == falloutTargets && viewItem == falloutTarget",
+                    "when": "view == fallout.build && viewItem == falloutTarget",
                     "group": "inline@2"
                 }
             ],
diff --git a/src/extension.ts b/src/extension.ts
index 7698e1f..5ae8279 100644
--- a/src/extension.ts
+++ b/src/extension.ts
@@ -141,7 +141,7 @@ export function activate(context: vscode.ExtensionContext): void {
     watcher.onDidDelete(() => provider.refresh());
 
     context.subscriptions.push(
-        vscode.window.registerTreeDataProvider('falloutTargets', provider),
+        vscode.window.registerTreeDataProvider('fallout.build', provider),
         watcher,
         vscode.commands.registerCommand('fallout.refreshTargets', refreshAll),
         vscode.commands.registerCommand('fallout.runTarget', (item?: TargetItem) => {