Skip to content

fix: close redundant PRs + friendly error on all design commands (v0.15.8.1)#817

Merged
garrytan merged 3 commits intomainfrom
garrytan/close-redundant-prs
Apr 5, 2026
Merged

fix: close redundant PRs + friendly error on all design commands (v0.15.8.1)#817
garrytan merged 3 commits intomainfrom
garrytan/close-redundant-prs

Conversation

@garrytan
Copy link
Copy Markdown
Owner

@garrytan garrytan commented Apr 5, 2026

Summary

Community PR triage + error handling polish.

PR Triage (GitHub ops):

Code Changes:

  • Friendly OpenAI org error expanded to evolve.ts, iterate.ts, variants.ts, check.ts. Previously only generate.ts showed a user-friendly message when the org wasn't verified.
  • >128KB regression test for Codex session discovery. Documents the buffer limitation so future Codex versions surface cleanly.

Test Coverage

All new code paths have test coverage (100%). Tests: 19 pass, 0 fail.

Pre-Landing Review

No issues found. Mechanical copy-paste of existing error handling pattern.

Plan Completion

All 8 plan items DONE. 2 intentionally deferred (Issue #778 browse auth, Issue #766 sub-agent context).

Reviews

Review Status
CEO Review CLEAR (supply chain trust audit)
Eng Review CLEAR (3 issues found, 3 resolved)
Codex Review CLEAR (7 findings, 5 tension points resolved)

Test plan

  • All bun tests pass (19 tests, 884 assertions)
  • Regression test for >128KB session_meta
  • Friendly error pattern verified across all 5 design files

🤖 Generated with Claude Code

garrytan and others added 3 commits April 5, 2026 01:53
Previously only generate.ts showed a user-friendly message when the
OpenAI org wasn't verified. Now evolve, iterate, variants, and check
all detect the 403 + "organization must be verified" pattern and show
a clear message with the correct verification URL.
Documents the current 128KB buffer limitation. When Codex embeds
session_meta beyond 128KB, this test will fail, signaling the need
for a streaming parse or larger buffer.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

E2E Evals: ✅ PASS

0/0 tests passed | $0 total cost | 12 parallel runners

Suite Result Status Cost

12x ubicloud-standard-2 (Docker: pre-baked toolchain + deps) | wall clock ≈ slowest suite

@garrytan garrytan merged commit 2b08cfe into main Apr 5, 2026
18 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