Remove arc-paste / share stack — revert to legacy planner - #57
Merged
Conversation
Delete the /share/[id] SvelteKit route and web/src/lib/paste/ client lib. Strip isShareRoute special-casing from the global layout so it always renders the normal app shell. Remove the Share settings section from the settings page. Revert the vite proxy target to the hardcoded default port.
…haned sharesconfig
Remove ShareConfig struct, Config.Share field, share.server validation, legacy share_author/share_server migration mapping, and arc config command handling for share.author/share.server keys.
Remove the `share` property and `ShareConfig` schema from the OpenAPI config schemas, and regenerate `openapi.gen.go` and `types.ts` to match the Go `Config` struct which no longer has a `Share` field.
Delete vitest.config.ts, remove vitest devDependency, and restore the test script to run playwright e2e — reverting the arc-paste foundation additions now that all paste/share test files are gone.
# Conflicts: # arc-paste/main.go # cmd/arc/config.go # cmd/arc/config_test.go # cmd/arc/share.go # internal/config/validate_test.go
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
1 participant
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 entire arc-paste / share stack (UI + standalone service + backend) and restores the legacy planner (
arc plan→/planner/[planId]) as the sole design-review surface. The share/paste feature was not well received; the legacy planner was, and it remained fully intact in the tree — so this is an additive-layer removal, not a code-restore.Net: 100 files, +329 / −11,472.
What was removed
/share/[id]route + components,lib/paste/client/crypto; stripped share branches from+layout.svelteand the settings page; reverted vite proxy; removed vitest (it existed solely for the paste/share unit tests)arc-paste/standalone binary + Caddyfile/compose/Dockerfile;internal/paste/cmd; Makefilebuild-paste/api/paste/*+/shareshandlers,arc shareCLI,internal/clientshare methods, server wiring (paste route reg, share routes, legacy shares.json import, sharesconfig factory); regenerated OpenAPI + TS typesStorageshare methods +ErrShareNotFound, SQLite adapter, sqlc schema/queries (regenerated),types.Share/ShareKind, paste engine,internal/sharesconfigShareConfigstruct/field/default/validation, legacyshare_*migration mapping,arc configshare handling, OpenAPIShareConfigschema018_drop_shares.sqldropsshares+paste_shares/paste_events/paste_migrations(forward-only;017left intact as a released migration)docs/runbooks/paste-server.md,docs/runbooks/review_howto.mdKept intact
Legacy planner:
cmd/arc/plan.go,internal/api/plans.go,internal/storage/sqlite/plans.go,web/src/routes/planner/, migrations004/013/014.CHANGELOG.mdand historicaldocs/plans/*are deliberately untouched (historical records).Verification
make build(frontend embedded),make test,make test-integration— greenbun lint(biome+eslint) +bun run check— greenarc-paste/share/paste/vitestreferences outsideCHANGELOGanddocs/plans/Follow-up (separate repo)
The arc plugin skills (
brainstorm/plan) still referencearc share. They live in a separate marketplace repo, so they're out of scope here and need a follow-up there to point only at the legacy planner.🤖 Generated with Claude Code