Skip to content

feat!: delete kix, quest.moq.dev, and quest runner - #575

Merged
kixelated merged 1 commit into
mainfrom
chore/delete-kix
Nov 8, 2026
Merged

kixelated merged 1 commit into
mainfrom
chore/delete-kix

Conversation

@kixelated

@kixelated kixelated commented Nov 8, 2026 •

Copy link
Copy Markdown
Collaborator

Quest serves moq-dev only, so kix goes. It had never run on a real repository: its GitHub App, kixel-quest, was installed nowhere. moq-bot, which the host's daemons already act as, is now the only App.

What's removed

  • The Worker in cloudflare/ (the decision inbox, PWA, push notifications, boards, sign-in, and job routing at quest.moq.dev), along with its Isolated nightly workflow and just worker-* recipes.
  • quest runner (src/runner/), the flake's runner package and its pinned Claude Code, opencode, CLIProxyAPI, and sandbox-runtime, the Sandbox workflow, just runner-sandbox, and the @opencode/client dependency.
  • The quest:kix hold label. quest:active is the only hold, so acquire's check for a rival hold label went too.
  • kix's sessions comment. quest review now reads the latest writer from the PR body's ## Models record alone, and quest usage makes one less API call per PR.
  • The site-only theme assets (theme.css and the PWA icons). design/theme.md keeps the CLI's colours, the logo, and the social image.
  • Quests that existed only for kix: m0/kix, m1/kix, open, launch, user-state, theme, m2/cloud, m2/automation, donor-runner, donor-isolation, and mobile-app. m0, m1, and m2's goals drop kix, and m0's Plan records the decision. Two cosmetic CLI items from the theme epic are dropped with it: pluralising quest check's "documents checked" and the column alignment for wide characters.

The land tests' fake GitHub moves from cloudflare/dev/github.ts to test/github.ts. It's cut from 1629 to 1135 lines by removing the paths that only the Worker exercised: SPDX, user tokens, and the inbox and job queries. The land tests reach the rest.

Public API

Breaking:

  • quest runner is gone.
  • The flake no longer exports packages.runner. No consumer used it: kixelated/nixos uses packages.default.
  • quest/core no longer exports KIX_LABEL.
  • A quest:kix label no longer holds a PR.

Nothing to migrate, since nothing used them.

Validation

just check, just test (1141 passed), and nix flake check pass locally.

After merge (operator)

  • Delete the quest.moq.dev Worker, its D1 database and R2 bucket, and its DNS record.
  • Delete the kixel-quest App (ID 5622964) from moq-dev's developer settings.

Models

  • Size: M
  • Implementer: Claude Code, claude-opus-5-5

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

Review

Rewritten by quest review after each round.

  • Reviewer: Codex, gpt-6.1-sol, medium effort
  • Session: 01a1bbff-992d-7952-8532-5a6269c5f45b, round 1
  • Head: 7827872
  • Base: origin/main at 19c6602
  • Score: 9/10, The scoped diff consistently removes kix and its integrations, with no actionable regressions found by code inspection; builds and tests were not run, as instructed.
  • Findings: none

Quest serves moq-dev only, and kix had never run on a real repository:
its App, kixel-quest, was installed nowhere. moq-bot, which the host's
daemons already act as, is the only GitHub App.

Removed: the Worker (cloudflare/), `quest runner` (src/runner/) and its
flake package and agent pins, the quest:kix hold label, kix's sessions
comment that quest review and quest usage read, the Isolated and Sandbox
workflows, the site-only theme assets, and the quests that built on kix.
The land tests' fake GitHub moves to test/github.ts, cut to what they use.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pullfrog

pullfrog Bot commented Nov 8, 2026

Copy link
Copy Markdown

Pullfrog  | Review this ➔

@kixelated
kixelated marked this pull request as ready for review November 8, 2026 16:31

@kixelated kixelated left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Automated review by review (OpenAI)

Reviewed head 7827872d898a7d666e0a65d6ad9178b7334de5b0 against base 19c6602e4a137e96b974126c65b5e66e04def3c7.

No actionable correctness regressions found in the retained CLI changes. I checked the removal boundaries and remaining source/test imports, the single-label hold policy and its tests, PR-body-only model selection and usage attribution, the extracted Claude rate-limit reader against its previous implementation, the relocated land-test GitHub fake, and package/Nix/workflow changes. The rate-limit extraction preserves rejected-event, billing-error, and legacy reset handling.

Direction: the code is consistent with the stated removal scope; no blocking code change identified. The Worker, storage, DNS, and GitHub App cleanup listed in the PR remains a separate operator step, not something this diff performs.

Static GitHub-only review. I did not run builds, tests, Nix checks, or live CLI flows, and have not independently verified the author's reported test results or external deployment state. This is a comment, not a merge approval.

(Written by review (OpenAI))

@kixelated
kixelated added this pull request to the merge queue Nov 8, 2026
Merged via the queue into main with commit 7fe9793 Nov 8, 2026
19 checks passed
@kixelated
kixelated deleted the chore/delete-kix branch November 8, 2026 16:42
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