chore: remove spec-kit tooling, historical specs, and migrate dev docs to smee.io#214
Merged
Merged
Conversation
…s to smee.io Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
Important Review skippedToo many files! This PR contains 180 files, which is 30 over the limit of 150. To get a review, narrow the scope: ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (180)
You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
…ync) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Removes the previously externally-synced spec-kit toolchain and its historical generated artifacts, and updates local development documentation + scripts to use the already-present dev:smee webhook relay (including a PORT-derived relay target) instead of referencing a non-existent dev:ngrok.
Changes:
- Deleted spec-kit-managed tooling/assets (
.specify/, speckit agents/prompts/skills) and historicalspecs/documents, plus cleaned up residual references in configs/comments. - Migrated local-dev webhook documentation from
dev:ngrokguidance todev:smee/smee.io relay usage. - Fixed the dev smee relay target to derive from
PORT, and ensured thedev-up.shharness exports a consistentPORTfor all child processes.
Reviewed changes
Copilot reviewed 135 out of 180 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| test/workflows/dispatcher.test.ts | Removes dead specs/… citation from test header comment. |
| src/workflows/dispatcher.ts | Removes dead specs/… citation from dispatcher doc comment. |
| src/webhook/events/issues.ts | Removes dead specs/… citation from label-dispatch protocol comment. |
| src/orchestrator/job-queue.ts | Removes dead specs/… citation from queue schema comment. |
| src/orchestrator/job-dispatcher.ts | Removes dead specs/… citation from scoped job envelope comment. |
| src/db/queries/dispatch-stats.ts | Removes dead specs/… citation from module header comment. |
| src/db/migrations/005_workflow_runs.sql | Removes dead specs/… reference from migration header. |
| src/daemon/workflow-executor.ts | Rewords doc comment to remove dead spec reference. |
| scripts/em-dash-sweep.ts | Removes skip rules/comments for deleted specs/ + spec-kit-managed paths. |
| scripts/dev-up.sh | Exports PORT (default 3030) so dev server/daemon/smee relay agree. |
| scripts/dev-smee.ts | Derives relay target from PORT (overrideable by SMEE_TARGET). |
| scripts/check-no-destructive-actions.ts | Removes dead specs/… reference in scoped-executor comment. |
| eslint.config.mjs | Drops specs/** ignore now that specs are removed. |
| docs/operate/setup.md | Rewrites local dev webhook docs to use dev:smee/smee.io relay model. |
| docs/operate/github-app.md | Updates GitHub App local-dev webhook URL guidance to smee.io + dev:smee. |
| docs/build/conventions.md | Updates em-dash gate docs to remove spec-kit/specs skip references. |
| CONTRIBUTING.md | Updates contributor docs to reference dev:smee instead of dev:ngrok. |
| CLAUDE.md | Removes now-dead SPECKIT marker block. |
| .vscode/settings.json | Removes speckit chat prompt/tooling configuration entries. |
| .prettierignore | Removes spec-kit-specific ignores; retains relevant managed paths. |
| .gitleaks.toml | Removes specs/ allowlist entry now that specs are removed. |
| .gitignore | Removes spec-kit ephemeral ignore rules (no longer applicable). |
| .github/workflows/research.yml | Removes specs/… references and updates docs-focus paths to docs/. |
| .env.example | Documents SMEE_TARGET override and its default target derivation. |
| specs/20260429-212559-ship-iteration-wiring/quickstart.md | Deleted historical spec artifact. |
| specs/20260429-212559-ship-iteration-wiring/data-model.md | Deleted historical spec artifact. |
| specs/20260429-212559-ship-iteration-wiring/contracts/ws-messages.md | Deleted historical spec artifact. |
| specs/20260429-212559-ship-iteration-wiring/contracts/job-kinds.md | Deleted historical spec artifact. |
| specs/20260429-212559-ship-iteration-wiring/checklists/requirements.md | Deleted historical spec artifact. |
| specs/20260427-201332-pr-shepherding-merge-ready/contracts/resolve-thread-mutation.md | Deleted historical spec artifact. |
| specs/20260427-201332-pr-shepherding-merge-ready/contracts/probe-graphql-query.md | Deleted historical spec artifact. |
| specs/20260427-201332-pr-shepherding-merge-ready/contracts/mcp-resolve-thread-server.md | Deleted historical spec artifact. |
| specs/20260427-201332-pr-shepherding-merge-ready/checklists/requirements.md | Deleted historical spec artifact. |
| specs/20260421-181205-bot-workflows/quickstart.md | Deleted historical spec artifact. |
| specs/20260421-181205-bot-workflows/contracts/workflow_runs.sql | Deleted historical spec artifact. |
| specs/20260421-181205-bot-workflows/contracts/webhook-dispatch.md | Deleted historical spec artifact. |
| specs/20260421-181205-bot-workflows/contracts/registry.schema.ts | Deleted historical spec artifact. |
| specs/20260421-181205-bot-workflows/contracts/handoff-protocol.md | Deleted historical spec artifact. |
| specs/20260421-181205-bot-workflows/checklists/requirements.md | Deleted historical spec artifact. |
| specs/20260415-000159-triage-dispatch-modes/contracts/triage-response.schema.json | Deleted historical spec artifact. |
| specs/20260415-000159-triage-dispatch-modes/contracts/shared-runner-internal.md | Deleted historical spec artifact. |
| specs/20260415-000159-triage-dispatch-modes/contracts/dispatch-telemetry.md | Deleted historical spec artifact. |
| specs/20260415-000159-triage-dispatch-modes/checklists/requirements.md | Deleted historical spec artifact. |
| specs/20260413-191249-daemon-orchestrator-core/checklists/requirements.md | Deleted historical spec artifact. |
| specs/20260410-164348-scheduled-research-workflow/contracts/workflow-inputs.md | Deleted historical spec artifact. |
| specs/20260410-164348-scheduled-research-workflow/contracts/labels.md | Deleted historical spec artifact. |
| specs/20260410-164348-scheduled-research-workflow/contracts/issue-body.md | Deleted historical spec artifact. |
| specs/20260410-164348-scheduled-research-workflow/checklists/requirements.md | Deleted historical spec artifact. |
| specs/20260409-081113-project-housekeeping/research.md | Deleted historical spec artifact. |
| specs/20260409-081113-project-housekeeping/quickstart.md | Deleted historical spec artifact. |
| specs/20260409-081113-project-housekeeping/data-model.md | Deleted historical spec artifact. |
| specs/20260409-081113-project-housekeeping/checklists/requirements.md | Deleted historical spec artifact. |
| .specify/workflows/workflow-registry.json | Deleted spec-kit workflow registry artifact. |
| .specify/workflows/speckit/workflow.yml | Deleted spec-kit workflow definition. |
| .specify/templates/spec-template.md | Deleted spec-kit template. |
| .specify/templates/plan-template.md | Deleted spec-kit template. |
| .specify/templates/constitution-template.md | Deleted spec-kit template. |
| .specify/templates/checklist-template.md | Deleted spec-kit template. |
| .specify/templates/agent-file-template.md | Deleted spec-kit template. |
| .specify/scripts/bash/setup-plan.sh | Deleted spec-kit script. |
| .specify/scripts/bash/check-prerequisites.sh | Deleted spec-kit script. |
| .specify/integrations/speckit.manifest.json | Deleted spec-kit integration manifest. |
| .specify/integrations/cursor-agent/scripts/update-context.sh | Deleted spec-kit integration script. |
| .specify/integrations/cursor-agent/scripts/update-context.ps1 | Deleted spec-kit integration script. |
| .specify/integrations/cursor-agent.manifest.json | Deleted spec-kit integration manifest. |
| .specify/integrations/copilot/scripts/update-context.sh | Deleted spec-kit integration script. |
| .specify/integrations/copilot/scripts/update-context.ps1 | Deleted spec-kit integration script. |
| .specify/integrations/copilot.manifest.json | Deleted spec-kit integration manifest. |
| .specify/integrations/claude/scripts/update-context.sh | Deleted spec-kit integration script. |
| .specify/integrations/claude/scripts/update-context.ps1 | Deleted spec-kit integration script. |
| .specify/integrations/claude.manifest.json | Deleted spec-kit integration manifest. |
| .specify/integration.json | Deleted spec-kit integration selector. |
| .specify/init-options.json | Deleted spec-kit init options. |
| .specify/extensions/git/scripts/powershell/initialize-repo.ps1 | Deleted spec-kit git extension script. |
| .specify/extensions/git/scripts/powershell/git-common.ps1 | Deleted spec-kit git extension script. |
| .specify/extensions/git/scripts/powershell/auto-commit.ps1 | Deleted spec-kit git extension script. |
| .specify/extensions/git/scripts/bash/initialize-repo.sh | Deleted spec-kit git extension script. |
| .specify/extensions/git/scripts/bash/git-common.sh | Deleted spec-kit git extension script. |
| .specify/extensions/git/scripts/bash/auto-commit.sh | Deleted spec-kit git extension script. |
| .specify/extensions/git/README.md | Deleted spec-kit git extension docs. |
| .specify/extensions/git/git-config.yml | Deleted spec-kit git extension config. |
| .specify/extensions/git/extension.yml | Deleted spec-kit git extension definition. |
| .specify/extensions/git/config-template.yml | Deleted spec-kit git extension template. |
| .specify/extensions/git/commands/speckit.git.validate.md | Deleted spec-kit command doc. |
| .specify/extensions/git/commands/speckit.git.remote.md | Deleted spec-kit command doc. |
| .specify/extensions/git/commands/speckit.git.initialize.md | Deleted spec-kit command doc. |
| .specify/extensions/git/commands/speckit.git.feature.md | Deleted spec-kit command doc. |
| .specify/extensions/git/commands/speckit.git.commit.md | Deleted spec-kit command doc. |
| .specify/extensions/.registry | Deleted spec-kit extension registry. |
| .specify/extensions.yml | Deleted spec-kit extension config. |
| .github/copilot-instructions.md | Deleted spec-kit-only Copilot instructions file. |
| .github/prompts/speckit.analyze.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.checklist.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.clarify.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.constitution.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.git.commit.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.git.feature.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.git.initialize.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.git.remote.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.git.validate.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.implement.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.plan.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.specify.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.tasks.prompt.md | Deleted speckit prompt stub. |
| .github/prompts/speckit.taskstoissues.prompt.md | Deleted speckit prompt stub. |
| .github/agents/speckit.constitution.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.git.commit.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.git.feature.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.git.initialize.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.git.remote.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.git.validate.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.plan.agent.md | Deleted speckit agent definition. |
| .github/agents/speckit.taskstoissues.agent.md | Deleted speckit agent definition. |
| .claude/skills/speckit-taskstoissues/SKILL.md | Deleted speckit skill definition. |
| .claude/skills/speckit-plan/SKILL.md | Deleted speckit skill definition. |
| .claude/skills/speckit-git-validate/SKILL.md | Deleted speckit skill definition. |
| .claude/skills/speckit-git-remote/SKILL.md | Deleted speckit skill definition. |
| .claude/skills/speckit-git-initialize/SKILL.md | Deleted speckit skill definition. |
| .claude/skills/speckit-git-feature/SKILL.md | Deleted speckit skill definition. |
| .claude/skills/speckit-git-commit/SKILL.md | Deleted speckit skill definition. |
Co-Authored-By: Claude Opus 4.8 <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.
Summary
Removes the externally-synced spec-kit ("specify") tooling and all generated specs that were no longer in active use, migrates the local-dev webhook docs from the non-existent
dev:ngrokto the already-shipped smee.io relay, and fixes a port mismatch in the smee relay script. ~155 files deleted; no runtime behaviour changed.Diagram
flowchart LR SPK["spec-kit assets<br/>.specify, .claude/skills/speckit-*<br/>.github/agents+prompts/speckit.*<br/>specs/, copilot-instructions.md"]:::removed SME["dev webhook relay<br/>dev:ngrok to dev:smee docs<br/>dev-smee.ts target derives from PORT<br/>dev-up.sh exports PORT"]:::added CFG["config cleanup<br/>.gitignore .gitleaks.toml<br/>.prettierignore eslint.config.mjs<br/>.vscode/settings.json"]:::updated SRC["source + test comments<br/>dead specs/ path citations removed"]:::updated classDef removed fill:#c0392b,color:#ffffff classDef added fill:#27ae60,color:#ffffff classDef updated fill:#2980b9,color:#ffffffChanges
spec-kit removal
.specify/(engine, templates, scripts, constitution, integrations, workflows)specs/— 7 completed-feature spec/plan/tasks folders (historical, all shipped).claude/skills/speckit-*(14),.github/agents/speckit.*and.github/prompts/speckit.*(14 each),.github/copilot-instructions.md(was entirely the spec-kit block)<!-- SPECKIT -->block inCLAUDE.md,.specify/rule in.gitignore,^specs/allowlist in.gitleaks.toml,specs/**ignore ineslint.config.mjs, speckit chat blocks in.vscode/settings.json, spec-kit skip patterns inscripts/em-dash-sweep.tsanddocs/build/conventions.md,.specify/speckit entries in.prettierignoresrc/andtest/to drop now-deadspecs/...path citations while keeping intent; updated.github/workflows/research.ymlto point at surviving pathssmee.io docs migration
docs/operate/setup.md,docs/operate/github-app.md,CONTRIBUTING.mdto referencedev:smeeinstead of the non-existentdev:ngrok, explaining the smee channel relay model (GitHub posts to a smee channel, a local client relays to localhost; no inbound tunnel)smee relay port fix (surfaced by review)
scripts/dev-smee.tsderives its relay target fromPORT(default 3000) instead of a hardcoded:3030scripts/dev-up.shexportsPORT(default 3030) so the server and the smee child agree in both the manualbun run devflow (3000) and thedev-up.shflow (3030)SMEE_TARGETin.env.exampleWhy
The spec-kit tooling and historical specs were no longer used. The docs referenced a
dev:ngrokscript that never existed inpackage.json(the repo already shipsdev:smee), so the migration also fixes stale setup instructions.Verification
bun run typecheck— cleanbun run lint— 0 errorsbun run format— cleangit grepconfirms zero residual.specify/ speckit /specs/referencesdev-smee.ts+dev-up.shsyntax-checked🤖 Generated with Claude Code