Skip to content

feat: refactor brin to use api#46

Merged
homanp merged 3 commits into
mainfrom
feat/refactor-brin-to-use-api
Mar 3, 2026
Merged

feat: refactor brin to use api#46
homanp merged 3 commits into
mainfrom
feat/refactor-brin-to-use-api

Conversation

@homanp
Copy link
Copy Markdown
Contributor

@homanp homanp commented Mar 3, 2026

What

Refactors brin to be a wrapper around the brin REST api

Why

Separation of services

Test plan

  • Tests pass locally
  • Tested manually

homanp added 2 commits March 3, 2026 22:27
Remove all backend crates (api, worker, watcher, cve, common, seed),
migrations, and Dockerfiles. The CLI is now a pure client that calls
api.brin.sh — all scanning and analysis happens server-side in brin-core.

- Single command: brin check <origin>/<identifier>
- Supports --details (?details=true), --webhook <url>, --headers
- --headers prints X-Brin-{Score,Verdict,Confidence,Tolerance} only
- 26 tests covering parse_artifact and BrinClient (wiremock)
- Updated README to reflect new scope and usage
…, dotenvy)

BRIN_API_URL is read natively by clap from the environment — no .env
loading needed. No production code uses serde derives, tracing spans,
or tracing-subscriber.
@homanp homanp self-assigned this Mar 3, 2026
@homanp homanp merged commit 6f97b56 into main Mar 3, 2026
5 checks passed
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