Skip to content

docs(clade): say which rename leaves the uuid alone - #76

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/uuid-not-immortal
Nov 10, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
chore/uuid-not-immortal

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Comment-only follow-up to #75. In .machine_readable/CLADE.a2ml, "A rename is NOT a phase change" now reads "A prefixed-name (clade code) change is NOT a phase change": after #75 the comment says a move or rename mints a new uuid (gv-clade-index ADR 0007), and this sentence is about the clade-code name, which ADR 0007 leaves out of the remint rule. Paragraph re-wrapped at 80 columns; no data value changes.


Generated by Claude Code

Follow-up to the uuid-not-immortal comment correction. The CLADE status
comment now says a move or rename mints a new uuid (gv-clade-index ADR
0007), but the next sentence still said "A rename is NOT a phase change
... uuid and phase are untouched". That sentence means a prefixed-name
(clade code) change, which ADR 0007 leaves out of the remint rule, so
it now says so. Comment-only; no data value changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0136P7ZmtCj9Doze7s38qbz5
@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 that changing a prefixed name does not change the phase: the previous name is retained as an alias, while the UUID and phase remain unchanged.

The CLADE lifecycle comment now specifies that a prefixed-name change adds the old name to aliases[] and leaves the UUID and phase unchanged.

Changes: CLADE lifecycle

Layer / File(s) Summary
Prefixed-name change clarification
.machine_readable/CLADE.a2ml
The lifecycle comment distinguishes a prefixed-name change from a phase change. It states that the old prefixed name goes to aliases[], while the UUID and phase remain unchanged.

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

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 21339

The change clarifies lifecycle wording without changing data or behavior. No merge-blocking risk is identified.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title clearly describes the change to the clade-code rename wording.
Description check Passed The description explains the comment update, its UUID context, and that no data values change.
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: b2bc4c04-610c-4821-8064-eac5b01ca30b
Commits

Reviewing files that changed from the base of the PR and between a2348d1 and 21339fd.

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

  • GitHub Check: CodeRabbit
  • GitHub Check: analyze (actions, none)
  • GitHub Check: Julia 1.11 - ubuntu-latest
  • GitHub Check: Validate K9 contracts
  • GitHub Check: Validate DEED manifests
  • GitHub Check: Empty-linter (invisible characters)
  • GitHub Check: Groove manifest check
  • GitHub Check: Julia 1.10 - ubuntu-latest
  • GitHub Check: semgrep-cloud-platform/scan
Additional comments: 1

.machine_readable/CLADE.a2ml: 1

32-34: LGTM!

@linear-code
linear-code Bot marked this pull request as ready for review November 10, 2026 20:54
@hyperpolymath
hyperpolymath merged commit 9cb1298 into main Nov 10, 2026
13 checks passed
@hyperpolymath
hyperpolymath deleted the chore/uuid-not-immortal branch November 10, 2026 21:03
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