Skip to content

docs(skills): question whether a quest is still needed - #221

Merged
kixelated merged 1 commit into
mainfrom
audit-question-need
Oct 26, 2026
Merged

kixelated merged 1 commit into
mainfrom
audit-question-need

Conversation

@kixelated

Copy link
Copy Markdown
Collaborator

Summary

  • The audit skill now looks for obsolete quests: a problem that is no longer real, or a plan that a later change or decision has made the wrong solution.
  • The guide's Execution section now has agents confirm a quest is still needed before starting it, and propose rewriting or deleting it if not.

Asked for by the maintainer during a moq.pro quest audit, where a quest still planned an upstream API for a problem already handled locally.

Checks

just check and just test pass.

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

Audits look for quests whose problem is gone or whose plan a later change
overtook, and starting a quest first confirms it is still needed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-25T17:07:16.727302Z 5edf6e9 PR opened
🔒 Security Review ✅ Completed 2026-10-25T17:08:19.368989Z 5edf6e9 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@kixelated

Copy link
Copy Markdown
Collaborator Author

Landing via /quest-merge.

  • Head 5edf6e92089b4a83d18ebe700a968bc2c11707e4: CI passed (4/4), Codex code and security reviews completed with no findings, no unresolved threads.
  • quest check --base origin/main --branch audit-question-need passes (50 documents ok), run with a quest built from the base.
  • No changes made during merge. The maintainer approved this PR's edits to assets/AGENTS.md and assets/skills/ by picking the merge.
  • Lands before kixelated/quest#211, which also edits the guide's Execution section and will need to merge main afterwards.
  • Squash merge, per the repository's AGENTS.md.

(Written by Claude Opus 5.5)

@kixelated
kixelated merged commit c647c05 into main Oct 26, 2026
4 checks passed
@kixelated
kixelated deleted the audit-question-need branch October 26, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant