Skip to content

docs: Update STRUCTURE.md for unified pipeline test [doc-updater] - #558

Merged
jwbron merged 1 commit into
mainfrom
egg/doc-update-unified-pipeline
Feb 12, 2026
Merged

docs: Update STRUCTURE.md for unified pipeline test [doc-updater]#558
jwbron merged 1 commit into
mainfrom
egg/doc-update-unified-pipeline

Conversation

@james-in-a-box

Copy link
Copy Markdown
Contributor

Update documentation to reflect changes from #554 (Unify local-mode and issue-mode pipeline behavior).

Changes

  • STRUCTURE.md: Added test_unified_pipeline_behavior.py to the integration tests structure listing

Context

Commit 12cc374 added a new integration test file that verifies unified pipeline behavior across local and issue modes. This test file validates:

  • Phase-based file push restrictions (replacing mode-based blocking)
  • Contract creation with pipeline_id keys
  • Prefixed file paths for concurrent pipeline support
  • Phase boundary state commits
  • Container/agent tracking

The test file should be documented in the project structure for completeness.

Triggered by: #554

— Authored by egg

Update project structure documentation to include the new integration test
file for unified local/issue pipeline behavior (added in #554).

Authored-by: egg

@james-in-a-box james-in-a-box Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Reviewed the diff and verified the change against the codebase.

Verification:

  • test_unified_pipeline_behavior.py exists in integration_tests/local_pipeline/ on main branch
  • ✓ File was added by PR #554 (merged) but the STRUCTURE.md documentation was not updated at that time
  • ✓ Tree formatting is correct ( transition for last item)
  • ✓ Comment description accurately reflects the file's purpose (per the module docstring)
  • ✓ Alignment and style match surrounding entries

No issues found.

— Authored by egg

@james-in-a-box

This comment has been minimized.

@james-in-a-box

Copy link
Copy Markdown
Contributor Author
egg is addressing review feedback...

@james-in-a-box

Copy link
Copy Markdown
Contributor Author

No actionable feedback to address - the review found no issues with the change. PR is ready for merge.

— Authored by egg

@james-in-a-box

Copy link
Copy Markdown
Contributor Author

egg feedback addressed. View run logs

1 previous review(s) hidden.

@jwbron
jwbron merged commit 97bfa29 into main Feb 12, 2026
23 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