Skip to content

feat: backfill embeddings endpoint + goreleaser homebrew/cosign (RUN4.05, RUN4.10) - #54

Merged
egerev merged 4 commits into
mainfrom
feat/tech-debt-sprint-5
Apr 9, 2026
Merged

feat: backfill embeddings endpoint + goreleaser homebrew/cosign (RUN4.05, RUN4.10)#54
egerev merged 4 commits into
mainfrom
feat/tech-debt-sprint-5

Conversation

@egerev

@egerev egerev commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • POST /admin/backfill-embeddings: queries NULL embeddings, batch embeds, updates
  • Admin auth required (require_scope("admin"))
  • batch_size bounded 1-200, savepoints per row, RLS via SET LOCAL
  • Goreleaser: Homebrew tap (GoSync-Inc/homebrew-alaya) + cosign checksum signing

Items closed

  • RUN4.05 (P2): Backfill endpoint missing
  • RUN4.10 (P3): Goreleaser without Homebrew tap + cosign

Test plan

  • 6 tests (auth required, counts, empty, workspace filter, batch bounds)
  • Full suite: 683 passed, 18 skipped

🤖 Generated with Claude Code

egerev and others added 2 commits April 9, 2026 02:56
….05, RUN4.10)

- Add POST /admin/backfill-embeddings endpoint that queries vector_chunks WHERE embedding IS NULL, batches embed_texts, and updates rows; returns processed/failed/total counts
- Add get_embedding_service dependency for injection in admin router
- Register admin router in main.py
- Add brews (Homebrew tap) and signs (cosign) sections to .goreleaser.yaml

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add require_scope("admin") dependency to backfill-embeddings endpoint
- Restrict batch_size with Field(ge=1, le=200)
- Use begin_nested() savepoints per UPDATE to prevent transaction abort on partial failure
- SET LOCAL app.workspace_id when workspace_id is provided for RLS scoping

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@egerev
egerev merged commit aa98697 into main Apr 9, 2026
13 checks passed
@egerev
egerev deleted the feat/tech-debt-sprint-5 branch April 9, 2026 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant