Skip to content
This repository was archived by the owner on Feb 15, 2026. It is now read-only.

feat: make all domains configurable for self-hosting#11

Merged
NicolasRitouet merged 3 commits into
mainfrom
feat/self-hosting
Feb 2, 2026
Merged

feat: make all domains configurable for self-hosting#11
NicolasRitouet merged 3 commits into
mainfrom
feat/self-hosting

Conversation

@NicolasRitouet

Copy link
Copy Markdown
Member

Summary

  • Replace ~14 hardcoded keyway.sh domain references with env-driven config values
  • Add GITHUB_URL config for GitHub Enterprise Server support (separate from API URL)
  • Make CORS require explicit ALLOWED_ORIGINS instead of hardcoding keyway.sh
  • Make error base URL, email from addresses, and OAuth flows use centralized config
  • Make Vercel integration slug configurable via env var
  • Fix GitHub App service and VCS client to use configurable API/OAuth URLs

Test plan

  • grep -r "keyway\.sh" src/ shows no hardcoded refs outside of defaults
  • Set GITHUB_URL=https://github.example.com and verify OAuth URLs use it
  • Verify CORS works with custom ALLOWED_ORIGINS
  • Verify error response URLs use ERROR_BASE_URL config

🤖 Generated with Claude Code

NicolasRitouet and others added 3 commits February 1, 2026 23:26
Replace hardcoded keyway.sh domains with env-driven config values.
Add GITHUB_URL for GitHub Enterprise support. Make CORS, error URLs,
email addresses, and OAuth flows use centralized config. Make Vercel
integration slug configurable.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add optional chaining fallback so errors.ts works when config
isn't fully initialized (e.g., in CI test runs without env vars).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Config properties like config.github and config.app can be undefined
in test environments where vi.mock provides partial config objects.
Use optional chaining with defaults for all module-level constants
and function-level config accesses that could be undefined.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@NicolasRitouet NicolasRitouet merged commit 8c4cf5f into main Feb 2, 2026
3 checks passed
@NicolasRitouet NicolasRitouet deleted the feat/self-hosting branch February 2, 2026 21:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant