Skip to content

skills: remove dead daemon/Fly instructions, fix wrong commands, repair skills-registry generator#1426

Merged
jonastemplestein merged 2 commits into
mainfrom
review-sweep/skills-and-registry
Jun 10, 2026
Merged

skills: remove dead daemon/Fly instructions, fix wrong commands, repair skills-registry generator#1426
jonastemplestein merged 2 commits into
mainfrom
review-sweep/skills-and-registry

Conversation

@jonastemplestein

@jonastemplestein jonastemplestein commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Sweep over agent skills plus a repair of the skills-publishing pipeline, which has been silently emitting an empty registry since the skills directory moved.

Skill doc fixes

  • error-pulse: removed the iterate tool subscribe-slack-thread / get-current-session-id instructions — those tools were deleted with the daemon stack in Remove legacy OS1 stack (apps/os, daemon, sandbox) #1341 and no equivalent thread-subscription mechanism exists today. The skill now states that escalation-thread follow-up is manual.
  • architect:
    • Required MCP access now points at the general Cloudflare API MCP server (https://mcp.cloudflare.com/mcp) instead of the product-specific observability endpoint that .agents/skills/cloudflare-traces/SKILL.md and .agents/skills/debug-mcp-server/SKILL.md explicitly forbid.
    • Dropped the fly CLI from diagnosis tool access and de-Fly'd the monitoring-mode section (kept as a pointer to architect-monitoring, which now covers OS/Cloudflare health monitoring; the Fly stack is removed).
    • Replaced the dead iterate tool exec-ts 'await slack.chat.postMessage(...)' snippet with a plain instruction to post the summary to #monitoring, noting the daemon tooling is gone. There is no documented programmatic Slack-posting CLI in the repo today, so the instruction stays mechanism-agnostic.
  • architect-diagnose-performance: removed the "Fly signals (machine health/usage)" input.
  • playwriter-spec: fixed the run command — apps/os has an e2e script (no test:e2e) and the suite reads APP_CONFIG_BASE_URL (not OS_BASE_URL).
  • cloudflare-traces EXAMPLES.md: the os-preview-2 example now uses the dev/preview account id 376ef7ed81b0573f93524de763666c15 instead of the garple account.
  • debug-os-worker: apps/os/src/entry.workerd.ts no longer exists; replaced with apps/os/src/worker.ts.

Registry pipeline repair

  • apps/iterate-com/scripts/generate-skills-registry.mjs still read <repoRoot>/skills, which was moved to .opencode/skills in fix: Move skills to the correct place #1097, and swallowed the ENOENT — so the published .well-known/skills registry has been empty ever since. The generator now reads .opencode/skills and fails loudly if the directory is missing.
  • Regenerated apps/iterate-com/backend/generated/skills-registry.ts; it now contains the two publish: true skills (example-skill, playwriter-spec).
  • lint-staged.config.cjs had the same stale skills/** glob, so the auto-regeneration hook never fired on skill changes; updated to .opencode/skills/**.
  • .opencode/skills/AGENTS.md updated to describe the fixed pipeline.
  • example-skill kept publish: true and got an honest description/body as a minimal registry smoke-test skill (see flags below).

Checks

  • pnpm install, pnpm format (oxfmt), pnpm lint, pnpm typecheck — all green.
  • node apps/iterate-com/scripts/generate-skills-registry.mjs and pnpm --dir apps/iterate-com skills:generate — output contains both publish:true skills.
  • pnpm --dir apps/iterate-com build — builds successfully with the regenerated registry.

Skipped

  • Nothing skipped; all nine items re-verified against the code and applied. Did not touch .agents/skills/adding-a-new-durable-object-mixin/ or .agents/skills/drizzle-migrations/ (owned by other PRs).

Flags for reviewers

  • example-skill judgment call: I kept publish: true and rewrote it as an explicit registry smoke-test skill rather than unpublishing it — having one always-published skill gives the .well-known/skills endpoint a guaranteed non-empty payload to check against. Flip to publish: false if you'd rather not ship it.
  • Architect Slack reporting is now mechanism-agnostic: with the daemon tooling gone I found no replacement programmatic Slack-posting path in the repo, so the skill just says to post to #monitoring with whatever access the runtime has. If a real mechanism exists (or gets added), the skill should name it.
  • Merging this changes what iterate.com/.well-known/skills serves on next iterate-com deploy: from an empty list to example-skill + playwriter-spec.

🤖 Generated with Claude Code


Note

Low Risk
Documentation and generated registry only; no runtime app logic, though iterate-com deploy will change the public skills list from empty to two published skills.

Overview
Repairs the skills publishing pipeline after skills moved to .opencode/skills: the generator and lint-staged no longer point at the old skills/ path or silently emit an empty registry—it reads .opencode/skills, errors if that folder is missing, and the checked-in skills-registry.ts now lists example-skill and playwriter-spec. .well-known/skills will serve real content on the next iterate-com deploy.

Agent skill docs are updated to match the current stack: removed Fly / daemon references (iterate tool exec-ts, subscribe-slack-thread), architect now requires the general Cloudflare API MCP and mechanism-agnostic Slack posting, error-pulse documents manual thread follow-up, and small fixes cover playwriter-spec e2e command/env, debug-os-worker entry at worker.ts, cloudflare-traces example account id, and example-skill as an explicit registry smoke test.

Reviewed by Cursor Bugbot for commit bf50747. Bugbot is set up for automated code reviews on this repo. Configure here.

jonastemplestein and others added 2 commits June 10, 2026 12:37
…ir skills-registry generator

- error-pulse: drop deleted subscribe-slack-thread / get-current-session-id
  instructions (daemon stack removed in #1341); thread follow-up is manual now
- architect: point at the general Cloudflare API MCP server instead of the
  forbidden product-specific observability endpoint, drop Fly CLI access and
  the Fly monitoring section (stack removed), replace dead 'iterate tool
  exec-ts' Slack snippet with a plain reporting instruction
- architect-diagnose-performance: drop Fly signals input
- playwriter-spec: fix e2e invocation (APP_CONFIG_BASE_URL + pnpm e2e; there
  is no test:e2e script and OS_BASE_URL is not read)
- cloudflare-traces EXAMPLES: os-preview-2 lives in the dev/preview account,
  not garple
- debug-os-worker: entry.workerd.ts no longer exists; the worker entry is
  apps/os/src/worker.ts
- generate-skills-registry.mjs: skills moved to .opencode/skills in #1097 but
  the generator still read <repoRoot>/skills and silently emitted an empty
  registry on ENOENT; fix the path and fail loudly when the dir is missing.
  Regenerated registry now contains example-skill and playwriter-spec.
- skills AGENTS.md: document the fixed pipeline
- example-skill: honest description/body as a registry smoke-test skill

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…e removed skills/ dir

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@jonastemplestein jonastemplestein merged commit 410219d into main Jun 10, 2026
9 checks passed
@jonastemplestein jonastemplestein deleted the review-sweep/skills-and-registry branch June 10, 2026 12:02
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