chore(ci): remove db/connectors backwards-compat symlink (#568) #80

2026-02-28T22:14:03+01:00

jpmschweitzer commented

2026-02-28 22:14:03 +01:00

Summary

Remove the db/connectors → tooling/db/ backwards-compat symlink added in Sprint 21 (#274)
Migrate all functional references from db/connectors/ to tooling/db/ across the repo
Move whatsinagame/static/db/connectors/ to whatsinagame/static/tooling/db/ and update 20 kit template/skill/test files
Clean up duplicate permission patterns from .claude/settings.json
Historical docs (old sprint briefings, changelog, archived discussions) left unchanged

Acceptance

db/ directory contains only schema.sql (no connectors entry)
tooling/db/ commands work normally
No functional db/connectors/ references remain (only historical docs)
make ci server clippy failures are pre-existing on main (not related to this PR)

🤖 Generated with Claude Code

## Summary - Remove the `db/connectors → tooling/db/` backwards-compat symlink added in Sprint 21 (#274) - Migrate all functional references from `db/connectors/` to `tooling/db/` across the repo - Move `whatsinagame/static/db/connectors/` to `whatsinagame/static/tooling/db/` and update 20 kit template/skill/test files - Clean up duplicate permission patterns from `.claude/settings.json` - Historical docs (old sprint briefings, changelog, archived discussions) left unchanged ## Acceptance - `db/` directory contains only `schema.sql` (no `connectors` entry) - `tooling/db/` commands work normally - No functional `db/connectors/` references remain (only historical docs) - `make ci` server clippy failures are pre-existing on main (not related to this PR) 🤖 Generated with [Claude Code](https://claude.com/claude-code)

jpmschweitzer added 2 commits 2026-02-28 22:14:04 +01:00

chore(db): remove db/connectors backwards-compat symlink (#568 ) b025fe8152

Remove the db/connectors → tooling/db/ symlink added in Sprint 21
(#274) and migrate all references to use tooling/db/ directly.

- Delete tracked symlink from db/connectors
- Remove duplicate db/connectors/* permission patterns from settings
- Update project-structure.md to reflect removal
- Move whatsinagame/static/db/connectors/ to whatsinagame/static/tooling/db/
- Update 20 whatsinagame template, skill, and test files
- Update comment references in client/tests/test_anti_tedium.gd
- Historical docs (old sprint briefings, changelog, discussions) left as-is

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

chore(meta): update changelog ded24de31b

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

jpmschweitzer commented

2026-02-28 23:06:01 +01:00

Review: ci → main (type: code)

Hoshe (QA): APPROVE

Symlink removed. All operational paths updated to tooling/db/. 80 remaining db/connectors hits are in archived docs (sprint briefings, discussions, workshops) — correct to leave as historical records. Zero db/connectors references remain in .claude/, decisions/, tooling/, Makefile, client/, server/, or whatsinagame/.

Tyre (Architecture): APPROVE

Clean scope. whatsinagame templates fully consistent. .claude/settings.json correctly updated. No cross-worktree risk — other branches still have the symlink until their next worktree-update, providing a graceful migration window.

Verdict: APPROVED

Ready to merge.

## Review: ci → main (type: code) ### Hoshe (QA): APPROVE Symlink removed. All operational paths updated to `tooling/db/`. 80 remaining `db/connectors` hits are in archived docs (sprint briefings, discussions, workshops) — correct to leave as historical records. Zero `db/connectors` references remain in `.claude/`, `decisions/`, `tooling/`, `Makefile`, `client/`, `server/`, or `whatsinagame/`. ### Tyre (Architecture): APPROVE Clean scope. whatsinagame templates fully consistent. `.claude/settings.json` correctly updated. No cross-worktree risk — other branches still have the symlink until their next `worktree-update`, providing a graceful migration window. ### Verdict: APPROVED Ready to merge.

jpmschweitzer closed this pull request

2026-02-28 23:15:11 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jpmschweitzer/settled-reach#80