Skip to content

quest: delete the fixer proxy quest, verified on the live host - #441

Open
kixelated wants to merge 2 commits into
mainfrom
quest/m0/host-land/fixer
Open

kixelated wants to merge 2 commits into
mainfrom
quest/m0/host-land/fixer

Conversation

@kixelated

@kixelated kixelated commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Deletes Connect the host fixer to CLIProxyAPI: its Goal is met on the live host, with no change needed.

  • The fixer reaches CLIProxyAPI with the gate's client key: kixelated/nixos#16 already loads gate-key into the fixer launcher through LoadCredential and passes OPENAI_BASE_URL=http://<slot>.1:8317/v1 and OPENAI_API_KEY into the container (scripts/quest-land-check.py). The Codex login and the Git/GitHub credentials stay outside it. Codex model turns in live fixer jobs show the maintainer's Codex login works through the proxy; the container holds no other model credential.
  • A real mechanical fix, through the queue watcher: on 2026-11-02 the watcher re-enqueued moq-dev/moq#7388 with the fixer's fix (a base merge, tested with cargo nextest: 4700 passed), and the PR merged at that head, bea3fab.
  • An unchanged requeue: the watcher re-enqueued moq-dev/quest#324 unchanged as flaky on 2026-11-03, and moq-dev/moq#7606 on 2026-11-02.

It drops the epic's Required entry and the matching "what remains" item in Gate host.

Choices made without asking

  • Counted the Codex land fixer model turns as proof of the Codex login rather than asking the maintainer to re-check /var/lib/cliproxyapi/oauth: the container has only the gate key and the proxy.

Validation

  • quest check --base origin/main and just check pass on fd56c19.

Review

Codex reviewer, gpt-6.1-sol at medium effort (quest review --base origin/main), session 01a19fbf-c315-7020-96ec-7c820230591d, round 1 on head fd56c19d273ab0b86a5f8ce06cf046890a9c0ad7: no findings.

Follow-ups

None.

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

kixelated and others added 2 commits November 2, 2026 20:51
The watcher re-enqueued moq-dev/moq#7388 with the fixer's mechanical fix,
which merged at that head, and re-enqueued #324 unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated kixelated changed the title Connect the host fixer to CLIProxyAPI quest: delete the fixer proxy quest, verified on the live host Nov 3, 2026
@kixelated
kixelated marked this pull request as ready for review November 3, 2026 04:52
@coderabbitai

coderabbitai Bot commented Nov 3, 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 5 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: 0fb99dc1-4c81-4d8e-a2c1-705528c58cb4
Commits

Reviewing files that changed from the base of the PR and between acb8682 and fd56c19.

Files selected for processing: 3
  • quest/m0/host-land/README.md
  • quest/m0/host-land/fixer.md
  • quest/m0/host-land/host.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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

This branch has not been deployed

No deployments
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