Skip to content

CL-5858: Library L2 list/get artifact HTTP routes - #48

Merged
TheGreatAxios merged 7 commits into
mainfrom
cl-5858-library-l2-http
Aug 10, 2026
Merged

CL-5858: Library L2 list/get artifact HTTP routes#48
TheGreatAxios merged 7 commits into
mainfrom
cl-5858-library-l2-http

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Tenant-scoped GET /api/tenants/:tenantId/artifacts (paginated list) and GET .../artifacts/:id over the mounted @corbits/artifacts engine.
  • Grant-checked via the existing asset resource family; degrades cleanly when artifacts mount is unavailable.
  • Unit tests cover happy path, empty list, get, missing id, and cross-tenant 404.

Test plan

  • bun test apps/hub/src/artifact-routes.test.ts (5 pass)
  • tsc -p apps/hub --noEmit
  • CI green

Linear: CL-5858

Pin the Scout-proven artifacts package and boot-mount it against
ARTIFACTS_DATABASE_URL so versioned artifact rows can share the hub
Postgres tenancy. Unset URL degrades to a no-op; Library HTTP routes
are not part of this change.
Wire list + get HTTP over the mounted @corbits/artifacts engine under
/api/tenants/:tenantId/artifacts, with grant checks on the asset family.
@TheGreatAxios
TheGreatAxios force-pushed the cl-5858-library-l2-http branch from e86cb6f to d9dc25e Compare August 10, 2026 00:28
Public-repo gate: env vars live in .env.example only; source comments
must not reference private sibling repos.
@TheGreatAxios
TheGreatAxios merged commit d4c1a9f into main Aug 10, 2026
2 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-5858-library-l2-http branch August 10, 2026 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant