Skip to content

coding: tell a later attempt which files an earlier attempt already changed (#377) - #380

Merged
DanielNetzer merged 1 commit into
mainfrom
forgecell/issue-377-6e239faf0dd6d096080e5d13
Nov 4, 2026
Merged

DanielNetzer merged 1 commit into
mainfrom
forgecell/issue-377-6e239faf0dd6d096080e5d13

Conversation

@DanielNetzer

Copy link
Copy Markdown
Owner

Implements issue #377.

Molecule: mol-377-0bca488ed1b2d9bd2682e0bd
Formula: forgecell-github-issue
Base: 56fb4c2cbe6e4c807e0e7eeec843812e92c11a67
Delivery: c6e06cc5f92926cd23532984c3b6a16c1e3b99a1

Awaiting exact-commit CI and human review. Harness-reported checks are not independent CI evidence. No merge, deployment or issue closure is authorized.

@DanielNetzer

Copy link
Copy Markdown
Owner Author

Adversarial review: no HIGH or MEDIUM findings. The note is added only for a revision after the first that has changed files from the pre-coding capture, and only for in-scope paths that pass ValidatePath. The paths are JSON-quoted, capped at 20, then 'and N more'. It's derived at run time, so the plan digest and the adapter protocol are unchanged, and the ledger records a note with the file count.

LOW: changed files outside the approved scope are filtered out of the list silently. Scope audit catches them anyway.

The branch is based on 56fb4c2; main has since gained only the docs-only #378.

@DanielNetzer
DanielNetzer marked this pull request as ready for review November 4, 2026 07:56
@DanielNetzer
DanielNetzer merged commit 1e2b187 into main Nov 4, 2026
2 checks passed
@DanielNetzer
DanielNetzer deleted the forgecell/issue-377-6e239faf0dd6d096080e5d13 branch November 4, 2026 08:12
Sign up for free to 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.

1 participant