Skip to content

Add CI coverage and route error boundaries#1

Merged
BunsDev merged 1 commit intomainfrom
okcode/codebase-improvements
Mar 27, 2026
Merged

Add CI coverage and route error boundaries#1
BunsDev merged 1 commit intomainfrom
okcode/codebase-improvements

Conversation

@BunsDev
Copy link
Copy Markdown
Contributor

@BunsDev BunsDev commented Mar 27, 2026

Summary

  • Added a GitHub Actions CI workflow that installs dependencies, runs secret scanning, and executes the test suite on pushes and pull requests targeting main.
  • Added route-level and app-level error boundaries for the main Next.js app sections so failures are contained and users get a consistent recovery path.
  • Expanded automated test coverage across API contracts, artifact extraction, auth rate limiting, database/migration behavior, ingestion state, memory entries, search, and SQL validation.
  • Updated core OpenTrust modules to support the new coverage and error-handling paths, including logging and safer auth/bootstrap flows.
  • Added a review document capturing implementation improvement recommendations.

Testing

  • pnpm test -- --run
  • pnpm exec secretlint "**/*"
  • Verified SQLite migration coverage with tests for table creation, idempotency, transactions, cascading deletes, and FTS search behavior.
  • Not run: browser-based manual verification of the new error boundary screens.

- Add Vitest coverage for core auth, DB, search, and memory flows
- Add Next.js error and not-found boundaries across app routes
- Wire up CI to run secret linting and tests
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
open-trust Ready Ready Preview Mar 27, 2026 6:11pm

@BunsDev BunsDev merged commit 514bea7 into main Mar 27, 2026
0 of 2 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