You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
docs(clade): uuid follows the GitHub path; qualify the prefixed-name case - #226
Comment-only follow-up. An earlier PR replaced "uuid is immortal" in the CLADE status comment with "uuid follows the forge path". This one makes it exact. gv-clade-index ADR 0007 (accepted 2026-11-10) derives the uuid from the GitHub path (UUIDv5 over github.com/<owner>/<name>): a move or rename on GitHub mints a new one, and a mirror move or a phase change never does.
A2ML is retired for new content. Correcting a factually wrong comment in an existing file is an edit, not new content.
Changes
.machine_readable/CLADE.a2ml: "uuid follows the forge path" → "uuid follows the GitHub path (a move or rename there mints a new one, gv-clade-index ADR 0007; a mirror move or a phase change never does)"; "A prefixed-name (clade code) change is NOT a phase change" → "A prefixed-name (clade code) change with the GitHub path unchanged is NOT a phase change". The paragraph is re-wrapped at 80 columns. No data values change. The owner approved this wording for the whole estate. It refines the wording merged earlier.
RSR Quality Checklist
Required
Tests pass (just test or equivalent): not run locally. Comment-only change, left to CI.
Code is formatted (just fmt or equivalent): n/a, no code
Linter is clean (no new warnings or errors): n/a, no code
No banned language patterns
No unsafe blocks without // SAFETY: comments
No banned functions
SPDX license headers present on all new/modified source files (not touched)
No secrets, credentials, or .env files included
Testing
The diff is one comment paragraph in one file. Every line is ≤80 characters. The commit is signed.
The uuid is UUIDv5 over github.com/<owner>/<name> (gv-clade-index ADR
0007), so say "GitHub path" rather than "forge path": a move or rename on
GitHub mints a new uuid, a mirror move does not. Scope the "not a phase
change" sentence to a prefixed-name change with the GitHub path unchanged.
Correcting a factually wrong comment in an existing A2ML file is an edit,
not new A2ML content.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136P7ZmTcNdC1jL1Vp3qW9H
No actionable comments were generated in the recent review.
Summary
Summary by CodeRabbit
Documentation
Clarified that UUID identity is based on the GitHub path. Moving a mirror or changing its lifecycle phase does not change the UUID.
Clarified that changing a prefixed name while keeping the same GitHub path records the previous name as an alias, without changing the UUID or phase.
The lifecycle-status comments now identify the GitHub path as the basis for UUID identity. They distinguish mirror moves from GitHub-path changes and clarify alias handling for prefixed-name changes. The changes update comments only; they do not change implementation.
The comments state that mirror moves and phase changes do not mint a UUID. If a prefixed name changes while the GitHub path stays the same, the old name goes to aliases[], with no UUID or phase change.
File: No configuration file from this repository applied
Commit: Unavailable
Layers: Organization UI
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 091da41c-9bc3-416b-b1df-c488ea0497d0
Commits
Reviewing files that changed from the base of the PR and between fd95f82 and c414d42.
Files selected for processing: 1
.machine_readable/CLADE.a2ml
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.
Follow-up: three more checks have failed on this PR: Validate K9 contracts, Groove manifest check and Validate DEED manifests. All three also fail on main (fd95f82), so this PR did not cause them either.
Follow-up: Empty-linter (invisible characters) has now failed on this PR. It also fails on main (fd95f82). I checked this PR's diff: the only non-ASCII character in it is the em dash (—), which was already in the line being replaced. There are no invisible characters, so the failure does not come from this PR.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Summary
Comment-only follow-up. An earlier PR replaced "uuid is immortal" in the CLADE status comment with "uuid follows the forge path". This one makes it exact. gv-clade-index ADR 0007 (accepted 2026-11-10) derives the uuid from the GitHub path (UUIDv5 over
github.com/<owner>/<name>): a move or rename on GitHub mints a new one, and a mirror move or a phase change never does.A2ML is retired for new content. Correcting a factually wrong comment in an existing file is an edit, not new content.
Changes
.machine_readable/CLADE.a2ml: "uuid follows the forge path" → "uuid follows the GitHub path (a move or rename there mints a new one, gv-clade-index ADR 0007; a mirror move or a phase change never does)"; "A prefixed-name (clade code) change is NOT a phase change" → "A prefixed-name (clade code) change with the GitHub path unchanged is NOT a phase change". The paragraph is re-wrapped at 80 columns. No data values change. The owner approved this wording for the whole estate. It refines the wording merged earlier.RSR Quality Checklist
Required
just testor equivalent): not run locally. Comment-only change, left to CI.just fmtor equivalent): n/a, no codeunsafeblocks without// SAFETY:comments.envfiles includedTesting
The diff is one comment paragraph in one file. Every line is ≤80 characters. The commit is signed.
Generated by Claude Code