Skip to content

fix(config): isolate custom config dir#25957

Open
jvanzyl wants to merge 17 commits intoanomalyco:devfrom
jvanzyl:devflow/hojo
Open

fix(config): isolate custom config dir#25957
jvanzyl wants to merge 17 commits intoanomalyco:devfrom
jvanzyl:devflow/hojo

Conversation

@jvanzyl
Copy link
Copy Markdown

@jvanzyl jvanzyl commented May 6, 2026

Summary

  • Honor OPENCODE_CONFIG_DIR as the config root without also scanning home .opencode.
  • Prevent Hojo runs configured with ~/.hojo from loading stale user-level OpenCode config.
  • Add a regression test covering custom config directory isolation.

Validation

  • bun test --timeout 30000 test/config/config.test.ts
  • bun run build
  • pre-push typecheck hook passed during branch push

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label May 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:compliance This means the issue will auto-close after 2 hours. needs:issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants