Skip to content

Effectify plugin agent regression test#25646

Merged
kitlangton merged 1 commit intodevfrom
kit/effectify-plugin-agent-test
May 3, 2026
Merged

Effectify plugin agent regression test#25646
kitlangton merged 1 commit intodevfrom
kit/effectify-plugin-agent-test

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • Convert the plugin-registered agent regression test to testEffect so it runs inside the Effect test runtime.
  • Use the real instance bootstrap layer so plugin config hooks still execute before Agent.list().
  • Dispose the loaded instance in the scoped test body before temp directory cleanup.

Tests

  • bun run test test/agent/plugin-agent-regression.test.ts
  • for i in 1 2 3; do bun run test test/agent/plugin-agent-regression.test.ts || exit $?; done
  • bun run test test/agent/agent.test.ts test/agent/plugin-agent-regression.test.ts
  • bun typecheck
  • push hook: bun turbo typecheck

@kitlangton kitlangton enabled auto-merge (squash) May 3, 2026 21:58
@kitlangton kitlangton merged commit c2b1974 into dev May 3, 2026
12 checks passed
@kitlangton kitlangton deleted the kit/effectify-plugin-agent-test branch May 3, 2026 22:07
terisuke added a commit to Cor-Incorporated/opencode that referenced this pull request May 4, 2026
Upstream's new provider.models hook (anomalyco#25646) filters models by ALLOWED_MODELS
before the guardrail allow-list applies. Without these entries, fork guardrails
that whitelist gpt-5.1-codex, gpt-5.1-codex-mini, gpt-5.1-codex-max, and
gpt-5.2-codex would be silently bypassed when the user is on ChatGPT OAuth.

Refs PR #196 review feedback (MEDIUM).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant