charles/claude-hooks

Fork 0

LBL-1: Schema + builtin-sync for label_catalog #754

New issue

Closed

opened 2026-05-02 17:27:12 +00:00 by claude-desktop · 1 comment

claude-desktop commented

2026-05-02 17:27:12 +00:00

Collaborator

Bundle directive

This story ships in one PR with #743 (AT-1) and #750 (SVC-1) — the foundation layer. Assign #743, not this issue. The PR closes all three (Closes #743 #750 #754).

As a platform engineer, I want a label_catalog table and a builtin-sync pass that mirrors config/labels.json into scope='builtin', so that the canonical label set per repo can move off the file system onto a DB-backed source of truth shared with the rest of the agent-config surface.

Acceptance criteria

Schema

migrations/<n>-label-catalog.ts creates label_catalog with the columns + scope check (builtin/global/repo) + UNIQUE(scope, repo, name) index per specs/config-to-db.md § Architecture.

Builtin sync

syncLabelCatalogBuiltin(labelsJsonPath) writes one builtin row per label. Idempotent.
getLabelCatalogForRepo(repo) resolves scope='repo' over scope='global' over scope='builtin'.

Tests

Unit: idempotent sync, scope ladder merge.
Integration: a scope='repo' row for area:dashboard shadows the builtin colour for that one repo only.

Out of scope

Swapping consumers — covered by LBL-2.
Dashboard CRUD UI — covered by LBL-3.

References

Spec: specs/config-to-db.md § Story LBL-1, § Architecture.
Bundled with: #743 (AT-1), #750 (SVC-1) — assign #743, this closes via the same PR.

## Bundle directive This story ships in **one PR with #743 (AT-1) and #750 (SVC-1)** — the foundation layer. **Assign #743**, not this issue. The PR closes all three (`Closes #743 #750 #754`). --- As a platform engineer, I want a `label_catalog` table and a builtin-sync pass that mirrors `config/labels.json` into `scope='builtin'`, so that the canonical label set per repo can move off the file system onto a DB-backed source of truth shared with the rest of the agent-config surface. ## Acceptance criteria ### Schema - [ ] `migrations/<n>-label-catalog.ts` creates `label_catalog` with the columns + scope check (`builtin`/`global`/`repo`) + UNIQUE(scope, repo, name) index per `specs/config-to-db.md` § Architecture. ### Builtin sync - [ ] `syncLabelCatalogBuiltin(labelsJsonPath)` writes one builtin row per label. Idempotent. - [ ] `getLabelCatalogForRepo(repo)` resolves `scope='repo'` over `scope='global'` over `scope='builtin'`. ### Tests - [ ] Unit: idempotent sync, scope ladder merge. - [ ] Integration: a `scope='repo'` row for `area:dashboard` shadows the builtin colour for that one repo only. ## Out of scope - Swapping consumers — covered by LBL-2. - Dashboard CRUD UI — covered by LBL-3. ## References - Spec: `specs/config-to-db.md` § Story LBL-1, § Architecture. - Bundled with: #743 (AT-1), #750 (SVC-1) — assign #743, this closes via the same PR.

claude-desktop added the

area:infra

type:user-story

labels

2026-05-02 17:28:59 +00:00

claude-desktop added this to the Config to DB milestone

2026-05-02 17:29:00 +00:00

claude-desktop referenced this issue

2026-05-02 17:40:04 +00:00

AT-1: Schema + builtin-sync for agent_type tables #743

claude-desktop referenced this issue

2026-05-02 17:40:22 +00:00

SVC-1: Schema + builtin-sync for service_config #750

code-lead referenced this issue from a commit

2026-05-02 17:56:01 +00:00

feat(agent-config): schema + builtin-sync for agent_type / service_config / label_catalog

code-lead referenced this issue

2026-05-02 17:56:25 +00:00

feat(agent-config): schema + builtin-sync for agent_type / service_config / label_catalog #761

charles referenced this issue from a commit

2026-05-02 18:11:02 +00:00

feat(agent-config): schema + builtin-sync for agent_type / service_config / label_catalog (#761)

claude-desktop referenced this issue

2026-05-02 19:16:48 +00:00

SVC-1: Schema + builtin-sync for service_config #750

claude-desktop commented

2026-05-02 19:16:57 +00:00

Author

Collaborator

Shipped via PR #761 (commit 28027b9) as part of the foundation bundle alongside #743 (AT-1) and #750 (SVC-1). Closing.

Coverage:

migrations/<n>-label-catalog.ts created with the (scope, repo, name) unique index
syncLabelCatalogBuiltin(labelsJsonPath) writes one builtin row per label, idempotent
getLabelCatalogForRepo(repo) resolves repo → global → builtin

Shipped via PR #761 (commit `28027b9`) as part of the foundation bundle alongside #743 (AT-1) and #750 (SVC-1). Closing. Coverage: - `migrations/<n>-label-catalog.ts` created with the `(scope, repo, name)` unique index - `syncLabelCatalogBuiltin(labelsJsonPath)` writes one builtin row per label, idempotent - `getLabelCatalogForRepo(repo)` resolves `repo → global → builtin`

claude-desktop closed this issue

2026-05-02 19:17:01 +00:00