Skip to content

chore(quest): finish the pin-bumps epic - #733

Merged
kixelated merged 2 commits into
mainfrom
quest/m0/pin-bumps/README
Nov 14, 2026
Merged

kixelated merged 2 commits into
mainfrom
quest/m0/pin-bumps/README

Conversation

@kixelated

@kixelated kixelated commented Nov 14, 2026 •

Copy link
Copy Markdown
Collaborator

Finishes quest/m0/pin-bumps/README.md by deleting it. No other file links to it.

Its Waiting item has cleared. kixelated/nixos#70 closed unmerged and kixel-ai/dotfiles#106 replaced it. The quest-land journal on the maintainer's host now shows bump lines on every pass since 2026-11-13.

The Goal's end-to-end run took place in two halves. No single release has done both, because no release has changed a stub yet. The maintainer settled that the halves count (see Decisions).

Validation

A merge to src/ cuts a release: every merge since #708 that changes what ships has a tag. That runs from v0.3.0 at c1c04e0 to v0.11.0 at 33f4629 (#725, src/). Test-only and t3.json merges got no tag, as intended (git tag --points-at over c1c04e0~1..origin/main).

The watcher opens and lands a bump to that release: after the watcher restarted on the release-following build (2026-11-13 23:05 PST), one pass opened chore: bump quest to v0.11.0 in every enrolled repository. These landed on their own:

Some went to drafts, as designed, because the repository's own quests fail v0.11.0's new later-milestone check (#710):

With quest init's refreshed stubs in the PR: the watcher's bump PRs that followed main carried the stub changes from #649 and #664, and landed:

#717 changed only which SHA bump() targets and how the PR is titled. The quest init / quest check --fix / git add --all path is the same code (git show of #717 on src/cli/land-bump.ts).

Since 2026-11-11, stubs (each skill's name and description, and the set of skills under assets/skills/) last changed in #664, before release-per-merge. So every release-following bump so far changed only flake.lock, plus any quest fixes.

quest check, quest check --base origin/main, just check, and just test pass.

Public API: none. Wire: none.

Decisions

  • Should the two halves above count as the Goal's end-to-end run? Settled by the maintainer in this comment: the release-to-bump runs of v0.11.0 changed only flake.lock, and the earlier main-following bumps landed refreshed stubs through the same code.
    • ✅ Delete the epic now
    • Rejected: keep it waiting for the first release that changes a skill's name, description, or the skill set
    • Rejected: skip the PR
  • Pick: ✅ /quest-merge once reviewed.

Choices made without asking

  • I left kixel-ai/rind#749 and moq-dev/moq#8685 to their repositories' own merge flow. They wait on those repositories' quests, not on the bump pass.
  • The first release-following pass moved consumers from 0b60de0 (main) back to v0.11.0 (33f4629). Only t3.json differs between them, so I filed nothing. This was a one-time switch from following main, and no consumer pins ahead of a release now.

Follow-ups

None.

Models

Size: XS
Implementer: Claude Code, claude-opus-5-5, unknown effort (arm A)
Implementer: Claude Code, claude-sonnet-5-5, unknown effort (iterate, XS, arm A)

🤖 Generated with Claude Code

Review

Rewritten by quest review after each round.

  • Head: cc35f96

  • Base: origin/main at f46721b

  • Risk: low, Deleting a plan under quest/ changes no behavior that users or other repositories rely on, and a revert fully restores the document.

  • Classified: low by Codex, gpt-6.1-sol, low effort, session 01a1db9b-e9d9-7890-b6f2-3fa4553863a4

  • Reviewer: Codex, gpt-6.1-sol, medium effort

  • Session: 01a1db9b-f9dc-7d63-8e6a-37a5bb33f5d2, round 1

  • Score: 10/10, The diff only removes the completed pin-bumps epic document, changes no executable behavior, and has no identifiable landing blockers.

  • Rounds: 1; first round 0; findings 0: fixed 0, refuted 0, open 0

  • Findings: none

  • Reviewer: Codex, gpt-6.1-sol, low effort, shadow

  • Session: 01a1db9b-f9dd-7d90-8553-7143639174f9, round 1

  • Score: 10/10, The diff only deletes the completed pin-bumps planning document, with no executable changes, remaining path references, or actionable regressions.

  • Rounds: 1; first round 0; findings 0: fixed 0, refuted 0, open 0

  • Findings: none

kixelated and others added 2 commits November 14, 2026 10:58
The watcher bumps every enrolled repository to each release and lands it,
and refreshes stubs in bump PRs; no release since #708 has changed a stub.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated kixelated changed the title Every merge reaches every repository that pins quest chore(quest): finish the pin-bumps epic Nov 14, 2026
@kixelated

Copy link
Copy Markdown
Collaborator Author

Maintainer decision (via /quest-run, 2026-11-14):

  • The two halves count as the epic's end-to-end run: ✅ delete the epic now (rejected: keep it waiting for the first release that changes a stub; skip the PR).
  • Pick: ✅ /quest-merge once reviewed.

(Written by Claude Opus 5.5)

@kixelated
kixelated marked this pull request as ready for review November 14, 2026 19:50
@coderabbitai

coderabbitai Bot commented Nov 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 11 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d02fb8dd-a7a3-4898-a63e-b1e2213ea19b
Commits

Reviewing files that changed from the base of the PR and between 0b60de0 and cc35f96.

Files selected for processing: 1

  • quest/m0/pin-bumps/README.md
  • 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

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes

Reviewed the full diff, which deletes one planning document.

  • Deleted quest/m0/pin-bumps/README.md: removes the completed pin-bumps epic. A repo-wide grep finds no remaining references to pin-bumps, and the PR body's validation matches the epic's Waiting item.

Pullfrog  | View workflow run | Using claude-sonnet-5-5 | 𝕏

@kixelated

Copy link
Copy Markdown
Collaborator Author

Recorded the maintainer decision in the PR body's ## Decisions: the epic is deleted now, and its end-to-end run counts as done.

(Written by Claude Opus 5.5)

@kixelated
kixelated added this pull request to the merge queue Nov 14, 2026
Merged via the queue into main with commit 8c1e4c1 Nov 14, 2026
29 checks passed
@kixelated
kixelated deleted the quest/m0/pin-bumps/README branch November 14, 2026 19:55
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