Skip to content

Comments

docs: Add CHANGELOG entry for .egg-state file namespacing [doc-updater]#896

Merged
jwbron merged 1 commit intomainfrom
egg/doc-update-changelog-namespacing
Feb 24, 2026
Merged

docs: Add CHANGELOG entry for .egg-state file namespacing [doc-updater]#896
jwbron merged 1 commit intomainfrom
egg/doc-update-changelog-namespacing

Conversation

@james-in-a-box
Copy link
Contributor

docs: Add CHANGELOG entry for .egg-state file namespacing [doc-updater]

Update CHANGELOG.md to document the .egg-state file namespacing feature from #895.

This change documents that agent output files and check results now use per-pipeline identifiers (issue number or pipeline ID) to prevent merge conflicts when concurrent pipelines work on the same repository.

Changes:

  • Added entry in "Changed" section documenting new namespaced file patterns
  • Noted backward compatibility via fallback to old paths

Triggered by: c4031084

Authored-by: egg

Copy link
Contributor

@egg-reviewer egg-reviewer bot left a comment

Choose a reason for hiding this comment

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

Review: CHANGELOG entry for .egg-state file namespacing

Verdict: No issues found.

This is a 1-line documentation change replacing the - N/A placeholder under ### Changed with a changelog entry for the .egg-state namespacing feature from PR #895.

Verification

Verified the changelog entry against the actual implementation in PR #895 (merged):

Claim Verified
Agent outputs use {identifier}-{role}-output.json Yes — shared/egg_contracts/orchestrator.py line 343
Example 871-coder-output.json is correct Yes — matches test assertions in test_handoffs_namespaced.py:36
Check results use {identifier}-implement-results.json Yes — orchestrator/routes/pipelines.py lines 4116-4117
Backward compatibility via fallback to old paths Yes — fallback logic in orchestrator.py lines 353, tested in test_orchestrator.py:78-98
Identifier is issue number (preferred) or pipeline ID (fallback) Yes — _pipeline_identifier() at pipelines.py:99-107

The entry is accurate, concise, and follows Keep a Changelog conventions.

No security, correctness, or design concerns — this is a documentation-only change.

— Authored by egg

@james-in-a-box

This comment has been minimized.

@james-in-a-box
Copy link
Contributor Author

egg is addressing review feedback...

@james-in-a-box
Copy link
Contributor Author

Review feedback addressed: The automated review found no issues — all claims in the CHANGELOG entry were verified as accurate. No code changes needed.

— Authored by egg

@james-in-a-box
Copy link
Contributor Author

egg feedback addressed. View run logs

1 previous review(s) hidden.

@jwbron jwbron merged commit c2e9143 into main Feb 24, 2026
22 checks passed
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