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 - #381
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 a GitHub move or rename results in a new UUID, while moving a mirror or changing a lifecycle phase leaves the UUID unchanged.
Clarified that changing a prefixed name without changing the GitHub path adds the previous name to the aliases, without changing the UUID or lifecycle phase.
The lifecycle-status comment now states that GitHub moves or renames mint a new UUID. It also states that mirror moves do not change the UUID, and that a prefixed-name change with an unchanged GitHub path updates aliases[] without changing the UUID or phase.
Changes: Lifecycle identity rules
Layer / File(s)
Summary
Document UUID and alias rules .machine_readable/CLADE.a2ml
The comment identifies the GitHub path as the basis for UUID identity. It distinguishes GitHub moves or renames from mirror moves, and specifies how a prefixed-name change affects aliases[], UUID, and phase.
hyperpolymath/boj-server#380: Changes the same lifecycle-status comment. This PR further specifies the GitHub path and when a prefixed-name change updates aliases[].
Merge Risk: Minimal · up to 8c0a6
The change clarifies lifecycle identity documentation without changing data values. No material merge risk is identified.
Pre-merge checks: 4
Check name
Status
Explanation
Title check
The title clearly identifies the change: it specifies that the UUID follows the GitHub path and qualifies the prefixed-name case.
Description check
The description includes the required Summary, Changes, RSR Quality Checklist and Testing sections. It explains the comment change and states that tests were not run locally and will be left to CI. Th…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Description check
Explanation
The description includes the required Summary, Changes, RSR Quality Checklist and Testing sections. It explains the comment change and states that tests were not run locally and will be left to CI. The checklist is mostly complete, and the missing Screenshots section is not required when no screenshots apply.
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
File: No configuration file from this repository applied
Commit: Unavailable
Layers: Organization UI
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 722b01ce-9957-4182-8d17-cb94141d649e
Commits
Reviewing files that changed from the base of the PR and between 84014d4 and 8c0a6b7.
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.
Recent review details
Context from checks skipped due to timeout: 13
GitHub Check: CodeRabbit
GitHub Check: Detect relevant changes
GitHub Check: Hypatia Neurosymbolic Analysis
GitHub Check: Detect relevant changes
GitHub Check: Detect relevant changes
GitHub Check: Bridge — node (unit + boot smoke)
GitHub Check: Bridge — bun (unit + boot smoke)
GitHub Check: Detect relevant changes
GitHub Check: Detect relevant changes
GitHub Check: Bridge — deno (unit + boot smoke)
GitHub Check: Detect relevant changes
GitHub Check: Detect relevant changes
GitHub Check: semgrep-cloud-platform/scan
Additional context used
Code guidelines: 1
.github/copilot-instructions.md — auto-discovered
Path-based instructions: 1
Source excerpt: State files (.a2ml) live in `.machine_readable/` ONLY, never the root.
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