chore(cleanup): remove obsolete dev artifacts + linuxserver fixtures (-99k lines)#260
Conversation
github-actions
Bot
added
documentation
…e fixtures
99k+ lines and 316 files removed. None of this content is referenced by
the running system; it is all artifacts from the old multi-agent dev
process and template-generation pipeline that has moved to sibling
repos.
Removed:
- .claude/reports/ 148 .md files (old multi-agent working memory)
- .claude/multi-agent/ 7 files (dev workflow that's been retired)
- .claude/commands/ 33 custom slash commands tied to old workflow
- .claude/agents/ 4 obsolete subagent definitions
- .claude/{CHANGES,SLASH,AUDIT,QUICK,SETUP,WORKFLOW,RECOMMENDED,README}*.md
- plugins/ 75 files (lives in streamspace-plugins repo)
- manifests/templates-generated/ 30 linuxserver-based templates
(lives in streamspace-templates repo)
- manifests/templates/browsers/firefox.yaml linuxserver fixture
- tests/fixtures/template-firefox.yaml linuxserver fixture
- tests/reports/ tracked test report templates
- scripts/generate-templates.py obsolete (templates moved)
- scripts/generate-from-catalog.py obsolete (templates moved)
- scripts/migrate-templates.sh obsolete (one-time migration)
- api/streamspace-api 96MB compiled binary
Salvaged to docs/historical/ (architecturally substantive, frozen):
V2_ARCHITECTURE, MULTI_CONTROLLER_ARCHITECTURE, MULTI_CONTROLLER_IMPLEMENTATION,
PLUGIN_SYSTEM_ANALYSIS, PHASE2_ARCHITECTURE, REFACTOR_ARCHITECTURE_V2,
SECURITY_HARDENING, SECURITY_IMPL_GUIDE, ENTERPRISE_FEATURES,
COMPETITIVE_ANALYSIS, COMBINED_HA_CHAOS_TESTING.
Updated:
- CLAUDE.md documentation layout reflects wiki/docs/historical split
- .gitignore ui/playwright-report, ui/test-results, tests/reports,
.zencoder, api/streamspace-api
Verified: api / k8s-agent / docker-agent / ui all build clean.
JoshuaAFerguson
force-pushed
the
chore/cleanup-sweep-phase-1
branch
from
3b75992 to
08ea71b
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 08ea71b21c
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - name: vnc | ||
| containerPort: 3000 | ||
| - name: selkies | ||
| containerPort: 8080 |
There was a problem hiding this comment.
This change sets the new chrome-selkies default template to containerPort: 8080, but session protocol detection in api/internal/api/handlers.go classifies port 8080 as Guacamole (and does not read spec.streamingProtocol). With baseImage: ghcr.io/streamspace-dev/chrome-selkies:latest, new sessions from this default app can be misdetected as guacamole and routed to /guacamole instead of the Selkies path, which breaks the bootstrap streaming flow.
Useful? React with 👍 / 👎.
1 participant
Summary
Phase 1 of the big cleanup — remove obsolete dev artifacts and template fixtures with no remaining purpose. 316 files / ~99 K lines removed. No code paths affected; all four modules build clean.
What's gone
The whole `.claude/` directory is now empty after the deletions and is no longer present in the tree.
Salvaged to `docs/historical/` (architecturally substantive, frozen)
`V2_ARCHITECTURE.md`, `MULTI_CONTROLLER_ARCHITECTURE.md`, `MULTI_CONTROLLER_IMPLEMENTATION.md`, `PLUGIN_SYSTEM_ANALYSIS.md`, `PHASE2_ARCHITECTURE.md`, `REFACTOR_ARCHITECTURE_V2.md`, `SECURITY_HARDENING.md`, `SECURITY_IMPL_GUIDE.md`, `ENTERPRISE_FEATURES.md`, `COMPETITIVE_ANALYSIS.md`, `COMBINED_HA_CHAOS_TESTING.md`.
Updates
Coming next (separate PRs)
Verified