Skip to content

Remove the Named residual's dangling reference (0.13.1) - #29

Merged
dsnger merged 1 commit into
mainfrom
fix-named-residual-ref
Sep 28, 2026
Merged

dsnger merged 1 commit into
mainfrom
fix-named-residual-ref

Conversation

@dsnger

@dsnger dsnger commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

The Named residual in CLAUDE.md §5 and in /dev-workflow:workflow-init's template ended with
"as the standing sentences above require". That list exists only in the loop-rule target text
(§F), not above the paragraph in either installed prompt, so the clause pointed nowhere. It is
removed from both copies; the sentence before it states the rule on its own. No rule changed.
dev-workflow 0.13.0 -> 0.13.1.

Gate B: one pass, full (spec + quality), both branches NO FINDINGS; closed on the zero-finding
exit. No story cited, so no evidence entry is owed.

cycle al2i9s67y7; floor 3 per none; hook reminder threshold absent
cycle al2i9s67y7; Gate B (passes 1, codex): Findings 0. Blockers 0. Majors 0.

Human exceptions: none

Summary by CodeRabbit

  • Documentation
    • Clarified the “Named residual” guidance in the workflow documentation and template by removing a closing clause that referred to sentences not present there.
    • Added a changelog entry noting the wording update and that the rule itself is unchanged.
  • Chores
    • Updated the dev-workflow plugin version to 0.13.1.

The Named residual in CLAUDE.md §5 and in /dev-workflow:workflow-init's template ended with
"as the standing sentences above require". That list exists only in the loop-rule target text
(§F), not above the paragraph in either installed prompt, so the clause pointed nowhere. It is
removed from both copies; the sentence before it states the rule on its own. No rule changed.
dev-workflow 0.13.0 -> 0.13.1.

Gate B: one pass, full (spec + quality), both branches NO FINDINGS; closed on the zero-finding
exit. No story cited, so no evidence entry is owed.

cycle al2i9s67y7; floor 3 per none; hook reminder threshold absent
cycle al2i9s67y7; Gate B (passes 1, codex): Findings 0. Blockers 0. Majors 0.

Human exceptions: none
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 8b14b9c5-42c9-4d74-85b4-2d68a89ea38d

📥 Commits

Reviewing files that changed from the base of the PR and between c20f82c and 95a6dee.

📒 Files selected for processing (6)
  • .context/codex-reviews/gate-b-quality-al2i9s67y7-pass-1.md
  • .context/codex-reviews/gate-b-spec-al2i9s67y7-pass-1.md
  • CLAUDE.md
  • plugins/dev-workflow/.claude-plugin/plugin.json
  • plugins/dev-workflow/CHANGELOG.md
  • plugins/dev-workflow/commands/workflow-init.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The Named residual statement is revised in CLAUDE.md and the workflow-init command. The changelog records the wording update, the plugin version changes to 0.13.1, and two review result files report no findings.

Changes

Named residual wording

Layer / File(s) Summary
Update prompt wording
CLAUDE.md, plugins/dev-workflow/commands/workflow-init.md
Both copies remove the clause that says the standing sentences require correcting the reminder.
Record the release update
plugins/dev-workflow/CHANGELOG.md, plugins/dev-workflow/.claude-plugin/plugin.json, .context/codex-reviews/*
The changelog documents the wording update, the plugin version changes to 0.13.1, and both review result files report zero findings.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 95a6d

The release removes a dangling reference from both prompt copies without changing the rule. The changelog and version match; no material merge risk remains.

Architecture Summary

Architecture risk: 🔵 Low · up to 95a6d

The change affects 2 systems.

Changed systems: plugins, CLAUDE.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — plugins (service) was modified; 3 changed files map to changed impact.
  • observed — CLAUDE.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in CLAUDE.md: The Named residual statement removes the trailing clause that says the standing sentences above require correcting the reminder.
  • observed — Modified behavior in plugins/dev-workflow/.claude-plugin/plugin.json: The plugin version is updated from 0.13.0 to 0.13.1.
  • observed — Modified behavior in plugins/dev-workflow/CHANGELOG.md: Adds the 0.13.1 changelog entry documenting removal of the dangling-reference clause from both prompt copies and stating that the preceding sentence stands alone and the rule is unchanged.
  • observed — Modified behavior in plugins/dev-workflow/commands/workflow-init.md: The “Named residual” sentence removes its assertion that the standing sentences above require correcting the reminder, while retaining that the specific overstatement is out of scope and corrected in the same change.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely identifies the main change: removing the dangling Named residual reference and updating to version 0.13.1.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.

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

A rabbit reads the words anew
The dangling clause hops out of view
Two prompts now match in what they say
A changelog marks the release day
Then off the rabbit bounds away

Comment @coderabbitai help to get the list of available commands.

@greptile-apps

greptile-apps Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low risk] Removes a dangling reference from documentation.

The PR appears safe to merge.

Summary

The PR removes a dangling reference from the Named residual in the repository policy and the shipped workflow-init template.

  • Bumps the plugin version to 0.13.1 and records the change in the changelog.
  • Adds separate clean-result records for the Gate B spec and quality reviews.

Reviews (1) · Last reviewed commit: "Remove the Named residual's dangling ref..."

@dsnger
dsnger merged commit c73ee0d into main Sep 28, 2026
3 checks passed
@dsnger
dsnger deleted the fix-named-residual-ref branch September 28, 2026 08:09
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.

1 participant