Skip to content

feat(commit): include Lore-id in success output#59

Open
c-ferrier wants to merge 1 commit into
Ian-stetsenko:mainfrom
c-ferrier:feat/lore-id-output
Open

feat(commit): include Lore-id in success output#59
c-ferrier wants to merge 1 commit into
Ian-stetsenko:mainfrom
c-ferrier:feat/lore-id-output

Conversation

@c-ferrier

Copy link
Copy Markdown
Contributor

Included the Lore-id in both text and JSON outputs for the commit command. This allows users and agents to immediately reference the newly created Lore-id without needing to run a separate log command.

Key Changes:

  • Text output now includes the Lore-id using a pipe separator (e.g., Commit created: <hash> | Lore-id: <id>).
  • JSON output includes a top-level loreId field for machine readability.
  • Support added for --amend --no-edit to consistently display the ID.
  • Updated all relevant unit tests.

Verified with 425 unit tests and manual demonstration.

Updated CommitBuilder to return the generated or reused Lore-id and modified the commit command to display it in both text and JSON outputs. This allows users and agents to immediately reference the ID without an extra lookup.

- Text format: 'Commit created: <hash> | Lore-id: <id>'
- JSON format: Added top-level 'loreId' field.

Lore-id: 63d9089b
Tested: All 425 unit tests passed
Tested: Verified text and JSON outputs manually
Confidence: high
Assisted-by: Gemini:CLI [lore-protocol]
@c-ferrier c-ferrier requested a review from Ian-stetsenko as a code owner May 19, 2026 13:32
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