Skip to content

Generic cleanup, dashboard, skills, completion of quests workflows #81

Merged
KjellKod merged 17 commits intomainfrom
quests
Mar 21, 2026
Merged

Generic cleanup, dashboard, skills, completion of quests workflows #81
KjellKod merged 17 commits intomainfrom
quests

Conversation

@KjellKod
Copy link
Copy Markdown
Owner

@KjellKod KjellKod commented Mar 21, 2026

Summary

  • Regenerated the portfolio dashboard (23 → 38 quests) by backfilling 7 journal entries for 10 merged PRs since Mar 9
  • Fixed Codex sandbox permission prompt by adding sandbox_permissions: "workspace-write" to all reviewer/fixer MCP invocations
  • Automated quest completion (journal + README + archive) with new scripts/quest_complete.py

Changes

  • 7 new journal entries in docs/quest-journal/ covering PRs Add Codex-led Claude bridge runtime and Quest workflow updates #68-Increase bridge timeout default from 90s to 30 minutes #80
  • docs/dashboard/index.html regenerated: 36 finished, 2 abandoned
  • .skills/quest/delegation/workflow.md: added sandbox_permissions: "workspace-write" to 5 Codex invocations (plan reviewer full/fast, code reviewer full/fast, fixer); clarified artifact write prompts; replaced manual Step 7 with quest_complete.py call
  • scripts/quest_complete.py (new): reads quest artifacts, generates journal entry with celebration_data JSON, updates README index, archives quest directory
  • .quest-manifest: added new script

Validation

  • scripts/validate-manifest.sh passes
  • Dashboard renders correctly with 38 quest cards (open docs/dashboard/index.html)
  • quest_complete.py --help loads without errors
  • Dry run of quest_complete.py against archived quest produces valid journal entry
  • GitHub Pages deployment triggers after merge (post-merge verification)

🤖 Generated with Claude Code

Dashboard was stale since Feb 21. Regenerated to include 7 quests
that were archived after that date plus journal entries for quests
that were never tracked in the dashboard.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
Journal entries were missing for 10 merged PRs (Mar 13-21).
Dashboard now shows 37 quests (35 finished, 2 abandoned) up from 23.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
…ompletion

Two fixes:

1. Add sandbox_permissions: "workspace-write" to plan reviewer, code
   reviewer, and fixer Codex MCP invocations. Only the builder had it,
   causing Codex to prompt for permission when reviewers tried to write
   artifact files. Also clarify prompt wording to distinguish artifact
   writes from source code modifications.

2. Add scripts/quest_complete.py to automate Step 7 of the quest
   workflow. The script reads quest artifacts, generates a journal entry
   with embedded celebration_data JSON, updates the README index, and
   archives the quest directory. Previously all of this was manual,
   causing completed quests to pile up unarchived with no journal entries.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
Dashboard now shows 38 quests (36 finished, 2 abandoned).

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 06:37 — with GitHub Actions Inactive
…te.py

Address Codex CI review findings:
- Exit non-zero if quest status is not 'complete' instead of warning
  and proceeding. The orchestrator only calls this after transition to
  complete, so hitting non-complete is a bug, not a judgment call.
- Raise FileExistsError if archive destination already exists instead
  of silently deleting it with shutil.rmtree.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 06:43 — with GitHub Actions Inactive
Address second-round Codex CI review findings:
- Collapse newlines and escape pipe characters in outcome text
  before interpolating into the README markdown table.
- Fail fast if docs/quest-journal/ is not found when walking up
  from the quest directory, instead of silently creating it in
  the wrong location.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
Check CI status immediately first, only sleep 180s if checks are
still pending. Avoids unnecessary waits when CI is already done
and avoids premature reads when CI takes longer than 60s.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 06:52 — with GitHub Actions Inactive
Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
Both quest_complete.py and quest_state.py were missing from the
manifest. quest_complete.py was added earlier; quest_state.py
(used by workflow for atomic state transitions) was still missing.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 06:58 — with GitHub Actions Inactive
Address Codex CI review findings:
- Move quest_complete.py (which archives) after summary and context
  health steps that read from .quest/<id>/logs/. Previously the
  archive happened first, making those paths invalid.
- Validate slug against [a-z0-9][a-z0-9-]* before using in file
  paths and markdown links.

Quest/Co-Authored by Claude Opus 4.6 (1M context), Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 07:04 — with GitHub Actions Inactive
…S.md

Surface KISS/DRY/YAGNI/SRP in README so the philosophy section tells
the full story: worldview, encoded principles, and why it matters.
Restructure AGENTS.md to separate mindset (Core Principles) from
mechanics (Change Discipline).

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 13:41 — with GitHub Actions Inactive
…ummary

Move journal directory discovery inside the skip_journal guard so
archive-only runs don't fail when docs/quest-journal/ is missing.
Track .quest_updated files during installation and list them
explicitly in the completion summary instead of a generic message.

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
…tant skill

Validation section now leads with principles (reduce ambiguity, cheapest
path first, separate tooling, real files, truthful claims) then rules
that flow from simple checkbox steps to detailed manual scenarios.
Softens negative-case requirement to "when meaningful" and uses
"scenario" framing to avoid redundant prerequisites across steps.

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
The --slurp --jq combination silently failed on some gh versions,
and 2>/dev/null swallowed the error, making dedup see zero existing
comments. Stream pages with --paginate --jq instead, build the JSON
array manually, and emit a warning on failure instead of silent fallback.

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 14:30 — with GitHub Actions Inactive
The previous approach wrapped output in [ ] brackets unconditionally,
so a failed gh api call with no stdout produced valid empty JSON [],
silently disabling dedup. Now uses set -o pipefail and checks the
pipeline exit status explicitly before validating JSON structure.

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 15:10 — with GitHub Actions Inactive
The claude mcp list call takes several seconds, leaving the user with
no feedback after "Codex CLI found". Show a status line so the pause
is expected.

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 15:43 — with GitHub Actions Inactive
@KjellKod KjellKod changed the title Regenerate portfolio dashboard with all 30 quests Generic cleanup, dashboard, skills, completion of quests workflows Mar 21, 2026
Merge redundant comprehension principle into Purpose paragraph, trim
the resolve-commands rule, and replace web/Node example with a
stack-neutral CLI example.

Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
@KjellKod KjellKod temporarily deployed to codex-ci-review March 21, 2026 16:33 — with GitHub Actions Inactive
@KjellKod KjellKod merged commit 4d8b170 into main Mar 21, 2026
6 checks passed
@KjellKod KjellKod deleted the quests branch March 21, 2026 22:56
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