Skip to content

test(opencode): lock BadRequest HttpApi error contracts - #10

Closed
TimothyVang wants to merge 1 commit into
mainfrom
agent/m13-httpapi-error-tests
Closed

test(opencode): lock BadRequest HttpApi error contracts#10
TimothyVang wants to merge 1 commit into
mainfrom
agent/m13-httpapi-error-tests

Conversation

@TimothyVang

Copy link
Copy Markdown
Owner

Summary

Test plan

  • bun typecheck in packages/opencode
  • monorepo pre-push bun turbo typecheck (29 packages)
  • bun test test/server/httpapi-public-openapi.test.ts test/server/httpapi-error-contracts-source.test.ts
  • bun test test/server/ (305 pass, 2 skip)

Notes

  • Prefer tests over endpoint churn; no TUI rebrand.
  • Public OpenAPI still intentionally strips 400 on POST /experimental/console/switch via normalizeLegacyOperation (legacy SDK surface).

Extend public OpenAPI regression tests so experimental, tui, instance,
event, and provider 400 contracts from #6#8 cannot silently drop.
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

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 commented Jul 9, 2026

Copy link
Copy Markdown

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.

@TimothyVang

Copy link
Copy Markdown
Owner Author

Superseded by #11 (same OpenAPI locks + error-contract runner).

@TimothyVang TimothyVang closed this Jul 9, 2026
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