Skip to content

Implement a --comment option for sourcemeta-one-index#689

Merged
jviotti merged 1 commit intomainfrom
comment-option
Mar 2, 2026
Merged

Implement a --comment option for sourcemeta-one-index#689
jviotti merged 1 commit intomainfrom
comment-option

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Mar 2, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti marked this pull request as ready for review March 2, 2026 19:26
@augmentcode
Copy link

augmentcode bot commented Mar 2, 2026

🤖 Augment PR Summary

Summary: Adds a new CLI option to persist an informational comment alongside generated index output.

Changes:

  • Introduced a --comment / -m option to sourcemeta-one-index.
  • When provided, writes the comment to comment.json in the staging/output directory (via write_json_if_different).
  • Adjusted the step numbering in index_main to account for the new “store comment” stage.
  • Added CLI regression tests to ensure comment.json is removed when the option is omitted on rebuild, and updated when changed on rebuild.

Technical Notes: The new artifact participates in the existing staging+hardlink workflow, so incremental rebuilds can update or delete comment.json deterministically via the output tracking/removal pass.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode 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 completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 4 files

@jviotti jviotti merged commit 9c75de0 into main Mar 2, 2026
6 checks passed
@jviotti jviotti deleted the comment-option branch March 2, 2026 19:41
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