Skip to content

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

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 #126. 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 #126 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 clade code does not constitute a phase change.
    • Specified that the previous clade code is retained as an alias when the code changes.

The lifecycle comment now clarifies that a prefixed-name (clade code) change is not a phase change. It states that the old prefixed-name goes to aliases[], while the UUID and phase remain unchanged.

Changes: Lifecycle guidance

Layer / File(s) Summary
Clarify the prefixed-name rule
.machine_readable/CLADE.a2ml
The lifecycle comment specifies that a prefixed-name change does not change the phase or UUID, and that the old prefixed-name goes to aliases[].

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

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to dda0e

The clarification preserves the documented UUID lifecycle rule and is ready to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title describes the clarification that a prefixed-name change does not change the UUID.
Description check Passed The description explains the comment change and its distinction from moves or renames that mint a new UUID.
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: 85dc2d2b-a10a-41d3-8dc8-a534f7870725
Commits

Reviewing files that changed from the base of the PR and between 73fd100 and dda0e05.

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

  • GitHub Check: CodeRabbit
  • GitHub Check: semgrep-cloud-platform/scan
  • GitHub Check: Julia 1.11 - windows-latest
  • GitHub Check: Runtime smoke (CPU)
  • GitHub Check: Julia 1.11 - macos-latest
  • GitHub Check: Julia 1.10 - ubuntu-latest
  • GitHub Check: Julia 1.11 - ubuntu-latest
  • GitHub Check: Documentation sanity
  • GitHub Check: Zig Backend
  • GitHub Check: Julia 1.10 - windows-latest
  • GitHub Check: Coprocessor strategy, resilience, and TPU/NPU/DSP/MATH strict tests
  • GitHub Check: Certificate integrity checks
  • GitHub Check: GPU fallback tests - macos-latest
  • GitHub Check: Idris2 ABI typecheck
  • GitHub Check: Proof assistant bundle checks
  • GitHub Check: CPU vs Zig parity + accelerated smoke
  • GitHub Check: SMT proofs (Z3)
  • GitHub Check: Julia 1.10 - macos-latest
  • GitHub Check: Interop smoke (PyTorch import + ONNX export)
  • GitHub Check: Roadmap could-baselines (packaging + optimization + telemetry)
  • GitHub Check: GPU fallback tests - ubuntu-latest
  • GitHub Check: Julia nightly (crypto-enabled) - ubuntu-latest
  • GitHub Check: Julia 1.10 (crypto-enabled) - ubuntu-latest
  • GitHub Check: Julia nightly - ubuntu
  • GitHub Check: Julia 1 (crypto-enabled) - ubuntu-latest
  • GitHub Check: Build docs (Documenter, with doctests)
  • GitHub Check: estate-rules
  • GitHub Check: Creusot contracts (crypto/)
  • GitHub Check: Detect relevant changes
  • GitHub Check: Groove manifest check
  • GitHub Check: Validate K9 contracts
  • GitHub Check: Validate DEED manifests
  • GitHub Check: Empty-linter (invisible characters)
  • GitHub Check: Analyze (actions)
Additional comments: 1

.machine_readable/CLADE.a2ml: 1

35-37: LGTM!

@linear-code
linear-code Bot marked this pull request as ready for review November 10, 2026 20:55
@hyperpolymath
hyperpolymath merged commit 3b7f9ab into main Nov 10, 2026
37 of 40 checks passed
@hyperpolymath
hyperpolymath deleted the chore/uuid-not-immortal branch November 10, 2026 20:56
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