Skip to content

feat(tags): add GET /api/v1/tags endpoint with auth - #30

Merged
SteakFisher merged 4 commits into
mainfrom
feature/tag-sync
May 11, 2026
Merged

feat(tags): add GET /api/v1/tags endpoint with auth#30
SteakFisher merged 4 commits into
mainfrom
feature/tag-sync

Conversation

@thedevyashsaini

Copy link
Copy Markdown
Member
  • Add authenticateHttpApiKey utility for HTTP Bearer token validation
  • Add handleListTags handler that queries tags via Drizzle
  • Register route in registerApiRoutes.ts
  • Add lifecycle log for the new endpoint

- Add authenticateHttpApiKey utility for HTTP Bearer token validation
- Add handleListTags handler that queries tags via Drizzle
- Register route in registerApiRoutes.ts
- Add lifecycle log for the new endpoint
@thedevyashsaini

Copy link
Copy Markdown
Member Author

@SteakFisher check 7f54b11 only, the next one's prettier + bun test removed

@thedevyashsaini

Copy link
Copy Markdown
Member Author

corresponding implementation

ScrawnDotDev/Scrawn.js#19
ScrawnDotDev/cli#1

- Add storage/db/postgres/helpers/tags.ts with listTags() helper
- Route handler now calls listTags() instead of direct DB query
- Consistent with existing helpers (users.ts, apiKeys.ts, metadata.ts)
- Add findApiKeyByHash() to helpers/apiKeys.ts
- authenticateHttpApiKey now uses the helper instead of direct DB query
- Domain logic (revoked, expired checks) stays in the auth util
@SteakFisher
SteakFisher merged commit d039fec into main May 11, 2026
1 of 2 checks passed
@SteakFisher
SteakFisher deleted the feature/tag-sync branch May 11, 2026 10:24
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.

2 participants