skills: remove dead daemon/Fly instructions, fix wrong commands, repair skills-registry generator#1426
Merged
Merged
Conversation
…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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
iterate tool subscribe-slack-thread/get-current-session-idinstructions — 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.https://mcp.cloudflare.com/mcp) instead of the product-specific observability endpoint that.agents/skills/cloudflare-traces/SKILL.mdand.agents/skills/debug-mcp-server/SKILL.mdexplicitly forbid.flyCLI from diagnosis tool access and de-Fly'd the monitoring-mode section (kept as a pointer toarchitect-monitoring, which now covers OS/Cloudflare health monitoring; the Fly stack is removed).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.apps/oshas ane2escript (notest:e2e) and the suite readsAPP_CONFIG_BASE_URL(notOS_BASE_URL).os-preview-2example now uses the dev/preview account id376ef7ed81b0573f93524de763666c15instead of the garple account.apps/os/src/entry.workerd.tsno longer exists; replaced withapps/os/src/worker.ts.Registry pipeline repair
apps/iterate-com/scripts/generate-skills-registry.mjsstill read<repoRoot>/skills, which was moved to.opencode/skillsin fix: Move skills to the correct place #1097, and swallowed the ENOENT — so the published.well-known/skillsregistry has been empty ever since. The generator now reads.opencode/skillsand fails loudly if the directory is missing.apps/iterate-com/backend/generated/skills-registry.ts; it now contains the twopublish: trueskills (example-skill,playwriter-spec).lint-staged.config.cjshad the same staleskills/**glob, so the auto-regeneration hook never fired on skill changes; updated to.opencode/skills/**..opencode/skills/AGENTS.mdupdated to describe the fixed pipeline.publish: trueand 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.mjsandpnpm --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
.agents/skills/adding-a-new-durable-object-mixin/or.agents/skills/drizzle-migrations/(owned by other PRs).Flags for reviewers
publish: trueand rewrote it as an explicit registry smoke-test skill rather than unpublishing it — having one always-published skill gives the.well-known/skillsendpoint a guaranteed non-empty payload to check against. Flip topublish: falseif you'd rather not ship it.#monitoringwith whatever access the runtime has. If a real mechanism exists (or gets added), the skill should name it.iterate.com/.well-known/skillsserves on next iterate-com deploy: from an empty list toexample-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 andlint-stagedno longer point at the oldskills/path or silently emit an empty registry—it reads.opencode/skills, errors if that folder is missing, and the checked-inskills-registry.tsnow listsexample-skillandplaywriter-spec..well-known/skillswill 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 atworker.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.