Skip to content

docs: track promoted provenance record for issue #576 - #577

Merged
drmoisan merged 1 commit into
mainfrom
chore/promote-576-provenance
Aug 21, 2026
Merged

docs: track promoted provenance record for issue #576#577
drmoisan merged 1 commit into
mainfrom
chore/promote-576-provenance

Conversation

@drmoisan

Copy link
Copy Markdown
Owner

docs: track promoted provenance record for issue #576

Summary

Why

Issue #545 reported two defects in config/blast-radius.json. PR #575 fixed the first (the unfit module map). The second — TaskMaster's root build files being absent from shared_surfaces — is not fixed, and #545's own text predicted that fixing the module map alone would convert it into a reported false negative. That is now the observed behavior, so the defect was split out to #576 rather than closed with #545.

The promotion tooling left its provenance record untracked. An untracked file under docs/features/potential/promoted/ is lost to any tree clean, which would break the link from #576 back to the potential entry and discard the recorded evidence. Committing it closes that gap.

What Changed

PathChange
docs/features/potential/promoted/2026-08-21-blast-radius-shared-surfaces-missing-root-build-files.mdAdded (97 lines) — promoted potential-bug record for #576

Verification

Completed

CheckResult
Changed-file classification1 file, .md only, under docs/ — no code, config, test, or workflow path in the diff
Working treeClean after commit; artifacts/ is gitignored so the PR-body artifact does not enter the diff
Branch topologyBranched from origin/main at a01bdbb0; merge-base equals base, so the PR is a single-commit fast-forward candidate

Not applicable

The C# toolchain (CSharpier, analyzers, nullable, MSTest) is not exercised by this change: the diff contains no *.cs, *.csproj, *.props, *.targets, *.config, or workflow file. CI runs regardless and is the gate for merge.

Backward Compatibility / Migration Notes

None. Adding a Markdown record under docs/ changes no public surface, build input, or runtime behavior.

Risks and Mitigations

RiskMitigation
The record's content could drift from #576 if the issue is later editedThe record is a point-in-time provenance artifact, not a live mirror; #576 remains the authority for current status

Rollback is git revert of the single commit, with no build or data implications.

Review Guide

Single file, single commit. Read the record's Actual Behavior and Logs / Screenshots sections — they carry the reproduction output and the control case that together establish the defect is real and is a false negative rather than a missing feature.

Follow-ups

  • Bug: blast-radius-shared-surfaces-missing-root-build-files #576 remains open and unfixed: add TaskMaster's root build files to shared_surfaces (candidates include coverage.config, Directory.Build.targets, TaskMaster.sln, .editorconfig, .globalconfig, dotnet-tools.json, .csharpierignore) plus a shared_surface_globs entry for .github/workflows/**, verifying each path exists before adding it.
  • .claude/rules/parallel-orchestration.md still narrates the reference repository's seven subsystem modules, which do not describe the shipped TaskMaster config. Noted in Bug: blast-radius-shared-surfaces-missing-root-build-files #576 as a related documentation defect.

GitHub Auto-close

  • None

This pull request does not fix#576; it only commits that issue's provenance record. The context bundle detects #576 as a referenced issue, not as an autoclose target, and no autoclose list was supplied. Emitting a closing keyword here would wrongly close an open, confirmed bug.

`potential_to_issue` wrote this promoted record when issue #576 was filed for
the second defect in issue #545 — TaskMaster's root build files are absent from
`shared_surfaces` in `config/blast-radius.json`, so two items that both edit
`coverage.config`, `Directory.Build.targets`, or `TaskMaster.sln` report
`conflict=False` and can be scheduled into the same parallel cohort.
The record is the provenance artifact linking #576 back to its potential entry
and carries the reproduction evidence gathered on `a01bdbb0`, including the
negative control that distinguishes a real fix from one that simply reports
`conflict=False` for every pair. Committing it so the audit trail survives; an
untracked file under `docs/features/potential/promoted/` is lost to any tree
clean.
No code, configuration, or test is changed by this commit.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016LdWAA7aMkzJ27NUW7WzaT
@drmoisan
drmoisan merged commit b9a9b92 into mainAug 21, 2026
5 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: blast-radius-shared-surfaces-missing-root-build-files

1 participant

@drmoisan