Skip to content

docs(clade): uuid follows the forge path, it is not immortal - #789

Merged
hyperpolymath merged 1 commit into
mainfrom
docs/clade-uuid-follows-forge-path
Nov 10, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
docs/clade-uuid-follows-forge-path

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

Comment-only correction. The CLADE status comment said the uuid is immortal. gv-clade-index ADR 0007 (accepted 2026-11-10) makes the uuid follow the forge path: a move or rename mints a new one, and a phase change never does.

The comment came from the A2ML-era backfill. Correcting a factually wrong comment in an existing file is an edit, not new content.

Changes

  • .machine_readable/CLADE.deed: "uuid is immortal" → "uuid follows the forge path (a move or rename mints a new one, gv-clade-index ADR 0007; a phase change never does)"; "A rename is NOT a phase change" → "A prefixed-name (clade code) change is NOT a phase change". The comment paragraph is re-wrapped at 80 columns. No data values change. The wording is the same as in the 97 correction PRs already merged across the estate.

Testing

The diff is one comment paragraph in one file. Every line is ≤80 characters. The commit is signed. tools/check-deed-gates.sh was not run locally, so it is left to CI.


Generated by Claude Code

The CLADE status comment said the uuid is immortal. gv-clade-index
ADR 0007 (accepted 2026-11-10) makes the uuid follow the forge path:
a move or rename mints a new one, and a phase change never does. A
prefixed-name (clade code) change still does not remint.

Comment-only correction. No data values change. A2ML is retired for
new content, but correcting a factually wrong comment in an existing
file is an edit, not new content.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136P7ZmTcNdC1jL1Vp3qW9H
@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Summary

Summary by CodeRabbit

  • Documentation
    • Clarified how UUIDs, names and phases are affected by forge moves, renames and phase changes.

The lifecycle-status comments now state that forge moves or renames mint a new UUID. They distinguish this from prefixed-name changes, which add the old name to aliases[] without changing the UUID or phase. Phase changes do not mint UUIDs.

Changes: Lifecycle status comments

Layer / File(s) Summary
Clarify lifecycle identity rules
.machine_readable/CLADE.deed
The comments distinguish forge moves or renames, prefixed-name changes, and phase changes. They also retain the statement that extinct can transition to active.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: metadatastician

Merge Risk: Minimal · up to 238fb

The change clarifies identity rules without altering data. No actionable merge-blocking risk is identified.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title clearly describes the UUID correction and matches the main change.
Description check Passed The description explains the comment correction and the UUID behaviour covered by the changeset.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • 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.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 63826545-4144-4cb3-84a1-145768e56ebd
Commits

Reviewing files that changed from the base of the PR and between a4d02e1 and 238fbc9.

Files selected for processing: 1

  • .machine_readable/CLADE.deed

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: 2

  • GitHub Check: CodeRabbit
  • GitHub Check: semgrep-cloud-platform/scan
Additional comments: 1

.machine_readable/CLADE.deed: 1

41-46: LGTM!

@sonarqubecloud

Copy link
Copy Markdown

@linear-code
linear-code Bot marked this pull request as ready for review November 10, 2026 23:05
@hyperpolymath
hyperpolymath merged commit df37279 into main Nov 10, 2026
9 checks passed
@hyperpolymath
hyperpolymath deleted the docs/clade-uuid-follows-forge-path branch November 10, 2026 23:12
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.

3 participants