Skip to content

Emit build-graph.json for editor tooling - #486

Merged
ChrisonSimtian merged 5 commits into
Fallout-build:mainfrom
ChrisonSimtian:emit-build-graph-json
Jul 13, 2026
Merged

Emit build-graph.json for editor tooling#486
ChrisonSimtian merged 5 commits into
Fallout-build:mainfrom
ChrisonSimtian:emit-build-graph-json

Conversation

@ChrisonSimtian

@ChrisonSimtianChrisonSimtian commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Adds SerializeBuildGraphAttribute (priority 20, ahead of --plan/--help) that writes <temp>/build-graph.json on build initialization — a machine-readable projection of the target graph for editor tooling (the VS Code extension's targets view + dependency graph).

  • Schema v1: { version, falloutVersion, targets[] }; each target carries name, description, declaredIn, default, listed, and dependsOn/after/triggeredBy/triggers.
  • Best-effort: wrapped in try/catch — never fails a build, only writes a temp file.
  • Reuses the same graph data as the --plan renderer; deterministic output (targets and relation lists sorted).

Restores an emitter that previously existed (a stale .fallout/temp/build-graph.json was still checked into the tree) but was no longer in source. Verified by dogfooding: regenerated output matches the prior file except for the advanced build height and a since-removed target.

🤖 Generated with Claude Code

@ChrisonSimtian@claude
Emit build-graph.json on build initialization for editor tooling
d031915
Add SerializeBuildGraphAttribute, which writes a machine-readable
projection of the target graph to <temp>/build-graph.json on every build
init: target names, descriptions, declaring type, default/listed flags,
and the four relation kinds (dependsOn/after/triggeredBy/triggers).
Reuses the same graph data as the --plan renderer. Runs at priority 20
so --plan and --help runs also refresh it. Best-effort: wrapped so a
serialization failure can never fail a build. Consumed by the VS Code
extension's targets view and dependency graph.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisonSimtian
ChrisonSimtian requested a review from a team as a code ownerJuly 13, 2026 03:11
@ChrisonSimtianChrisonSimtian added the target/vCurrent Targets the current version label Jul 13, 2026
dennisdoomen
dennisdoomen approved these changes Jul 13, 2026
View reviewed changes
Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
ITaluone
ITaluone suggested changes Jul 13, 2026
View reviewed changes

@ITaluoneITaluone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool! :)

Just these two little issues.

Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
@ChrisonSimtian

ChrisonSimtian commented Jul 13, 2026

Copy link
Copy Markdown
CollaboratorAuthor

Very cool! :)

Just these two little issues.

been cooking this for a while, when we release 10.4 i also intend to release a set of IDE extensions starting with VS Code

ChrisonSimtianand others added 2 commits July 13, 2026 17:49
@ChrisonSimtian@ITaluone
Update SerializeBuildGraphAttribute.cs
1958fae
Co-authored-by: ITaluone <44049228+ITaluone@users.noreply.github.com>
@ChrisonSimtian@ITaluone
Update SerializeBuildGraphAttribute.cs
4c99afd
Co-authored-by: ITaluone <44049228+ITaluone@users.noreply.github.com>
CopilotAI reviewed Jul 13, 2026
View reviewed changes

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a build-initialization extension that emits a machine-readable build-graph.json into the build temporary directory so editor tooling can consume target metadata and relationships.

Changes:

  • Registers a new build extension ([SerializeBuildGraph(Priority = 20)]) ahead of --plan/--help.
  • Introduces SerializeBuildGraphAttribute to serialize a deterministic, sorted schema-v1 target graph model to <temp>/build-graph.json.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

FileDescription
src/Fallout.Build/FalloutBuild.csRegisters the new build extension early in initialization.
src/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.csImplements best-effort JSON emission of the target graph for tooling consumption.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
Comment threadsrc/Fallout.Build/Execution/Extensions/SerializeBuildGraphAttribute.cs Outdated
@ChrisonSimtian
Potential fix for pull request finding
e8c6f35
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@ChrisonSimtianChrisonSimtian self-assigned this Jul 13, 2026
@ChrisonSimtian@claude
Extract build-graph projection into testable BuildGraphUtility
cb69ec3
Move the target-graph projection and JSON serialization out of
SerializeBuildGraphAttribute into a pure BuildGraphUtility, so the
build-graph.json contract the VS Code extension consumes can be
snapshot-tested without driving a full build. The attribute keeps only
the build-lifecycle hook and the file write.
Add BuildGraphUtilitySpecs: Verify snapshots that lock the emitted JSON
shape, plus unit coverage for target/dependency ordering, relation-kind
mapping, declaredIn, null handling, and version normalization.
Address review feedback: rename ResolveFalloutVersion -> FindFalloutVersion
(it can return null) and brace the null guard.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisonSimtian
ChrisonSimtian merged commit f407c7f into Fallout-build:mainJul 13, 2026
1 check passed
@ChrisonSimtian
ChrisonSimtian deleted the emit-build-graph-json branch July 13, 2026 08:09
@ChrisonSimtianChrisonSimtian added the enhancement New feature or request label Jul 13, 2026
@ChrisonSimtianChrisonSimtian added this to the v2026 milestone Jul 13, 2026
@ITaluone

ITaluone commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

been cooking this for a while, when we release 10.4 i also intend to release a set of IDE extensions starting with VS Code

Out of curiosity: do you have knowledge building Rider plugins? Would like to help with those, but cannot know where to start

This was referenced Aug 7, 2026
Open
Merged
Merged
Closed
Merged
Merged
Closed
Closed
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@dennisdoomendennisdoomendennisdoomen approved these changes
+1 more reviewer
@ITaluoneITaluoneITaluone approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@ChrisonSimtianChrisonSimtian

Labels

enhancementNew feature or requesttarget/vCurrentTargets the current version

Projects

None yet

Milestone

v2026

Development

Successfully merging this pull request may close these issues.

4 participants

@ChrisonSimtian@ITaluone@dennisdoomen