Skip to content

Refactor configuration and remove production test seams - #90

Merged
jasonhedman merged 3 commits into
mainfrom
json/remove-production-test-seams
Aug 31, 2026
Merged

Refactor configuration and remove production test seams#90
jasonhedman merged 3 commits into
mainfrom
json/remove-production-test-seams

Conversation

@jasonhedman

Copy link
Copy Markdown
Contributor

Summary

  • move environment validation to src/env.ts and simplify installation-secret and Blob authentication code
  • colocate the vault Open Graph asset with the vault route
  • remove mutable dependency bags, test-only factories, and injected defaults from production modules
  • allow module mocking in test files and mock auth, database, connector, Blob, and Kernel boundaries directly
  • document the production test-seam policy in AGENTS.md

Validation

  • pnpm check
  • DATABASE_URL=postgresql://build:build@127.0.0.1:5432/build BETTER_AUTH_URL=http://localhost:3000 pnpm build

…n-test-seams

# Conflicts:
#	src/auth/tests/auth-session.test.ts
#	tests/agent/hooks/session-owner.test.ts
#	tests/agent/subagents/worker/hooks/browser-trace-telemetry.test.ts
#	tests/agent/subagents/worker/tools/kernel-browser-contract.test.ts
@jasonhedman
jasonhedman merged commit 8a68c44 into main Aug 31, 2026
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