Skip to content

Build/Deployment/Run Configuration views + Explorer dock - #2

Draft
ChrisonSimtian wants to merge 1 commit into
developfrom
feature/views-and-run-config
Draft

Build/Deployment/Run Configuration views + Explorer dock#2
ChrisonSimtian wants to merge 1 commit into
developfrom
feature/views-and-run-config

Conversation

@ChrisonSimtian

Copy link
Copy Markdown
Contributor

Draft. Restructures the extension's single Targets view into several collapsible Views, mirrors the build targets into the file Explorer, and adds a Run Configuration form.

What's in it

  • Build view — the target tree, registered in both the ☢️ Fallout container (fallout.build) and the file Explorer (fallout.buildExplorer); one TreeDataProvider, two homes.
  • Deployment view — placeholder; lights up once the framework emits a deployment-graph.json for the ADR-0009 CD flow.
  • Run Configuration — a webview-view form: parameters (workspaceState → --name value args) and secrets (SecretStorage → env vars; values never rendered).
  • Run flow — ▶ applies the saved config silently; Run with Parameters… prompts an InputBox prefilled with the target + saved args.

Scope / relation to #1

Relates to #1. This delivers the Run Configuration view (the vessel); #1's real ask — declaration-driven population (e.g. NuGetApiKey appearing automatically from the build's [Parameter]/[Secret] members) — is not in this PR and stays open. It's blocked on Fallout-build/Fallout#499 (the graph must emit parameters[] first). So this PR does not close #1.

Verify

  • npm run compile clean; new src/runConfig.ts.
  • Load via Extension Development Host against a repo with .fallout/temp/build-graph.json.

@ChrisonSimtian ChrisonSimtian added this to the vscode extension milestone Jul 17, 2026
@ChrisonSimtian ChrisonSimtian added the enhancement New feature or request label Jul 17, 2026
@ChrisonSimtian ChrisonSimtian self-assigned this Jul 18, 2026
@ChrisonSimtian
ChrisonSimtian changed the base branch from main to develop August 14, 2026 08:14
@ChrisonSimtian

Copy link
Copy Markdown
Contributor Author

Closing and reopening to make CI run. This branch predates the PR gate, so no check ever ran on it, and retargeting to develop fires only pull_request: edited — which isn't in the default trigger set (opened, synchronize, reopened). Reopening is the least invasive fix: it triggers the gate without adding a commit to your branch.

Restructure the single Targets view into several collapsible Views in the
Fallout container, and mirror the build targets into the file Explorer.

- Build view: the existing target tree, registered under both fallout.build
  (Fallout container) and fallout.buildExplorer (Explorer dock) — one provider,
  two homes; refresh updates both.
- Deployment view: placeholder pending a deployment-graph.json from the CD model
  (ADR-0009); shows welcome content for now.
- Run Configuration: a webview-view form for parameters (workspaceState → CLI
  args) and secrets (SecretStorage → env vars, values never rendered).
- Run flow: ▶ applies the saved run config silently; "Run with Parameters…"
  prompts an InputBox prefilled with target + saved args before running.

Relates to #1 (this adds the Run Configuration view;
declaration-driven population of params/secrets from the build graph is the
remaining part, blocked on Fallout-build/Fallout#499).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisonSimtian

Copy link
Copy Markdown
Contributor Author

Decomposed into one issue per feature for the First full release milestone, so each can land as its own PR and produce its own release-note line:

This PR carries all six. Leaving it open and rebased — whether it merges as-is or gets split into six PRs is the housekeeping call for later. Splitting buys six release-note lines instead of one; merging as-is buys not doing the work.

Existing #1 (populate params/secrets from the build's declarations) is deliberately not in the milestone: it's blocked on Fallout#499, which is still open, and the current build-graph.json has no parameters[]. Putting it in would make the milestone hostage to another repo.

@ChrisonSimtian

Copy link
Copy Markdown
Contributor Author

Superseded by a per-feature split, so each feature lands as its own release-note line:

PR Issue Feature
#15 #9 Rename the Targets view to Build (breaking)
#16 #10 Explorer dock
#17 #11 Deployment view
#18 #12 Run Configuration — parameters
#19 #13 Run Configuration — secrets
#20 #14 Apply the run configuration on run

Stacked in that order, each based on the previous. Verified the six increments reconstruct this branch's tree byte-for-byte, and each one compiles on its own.

Leaving this open until the stack merges, as the umbrella.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Run Configuration: populate parameters & secrets from the build's declarations

1 participant