Feature/codex fable opus pipeline - #20
Conversation
There was a problem hiding this comment.
🎉 Thanks for your first PR to ORCH! We're excited to review your contribution.
Our CI will run automatically:
- ✅ TypeScript strict mode check (
tsc --noEmit) - ✅ Full test suite (1493+ tests via Vitest)
- ✅ ESM build verification
A maintainer will review your changes shortly. In the meantime, check the Contributing Guide to make sure everything is in order.
⭐ If you enjoy working with ORCH, a star on the repo goes a long way!
|
Thanks for the contribution. I can’t merge this PR in its current form. The PR mixes the Codex–Fable–Opus pipeline with extensive unrelated changes: it renames the package, disables npm publishing, rewrites repository and security links to the contributor’s fork, and changes existing adapters, goals, TUI, dependencies, and generated It is also behind Please split this into separate PRs:
All upstream package identity, npm publishing, repository links, and security-reporting links must remain unchanged. Please also remove unrelated fork-specific and generated changes. |
|
Thank you for the clear review. You were right that #20 mixed the workflow with unrelated package identity, publishing, repository/security-link, fork-specific, and generated I replaced it with two fresh PRs based directly on the current upstream
The replacement PRs preserve upstream package identity, npm publishing, repository/documentation/security-reporting links, and contain no generated |
What does this PR do?
Type of change
Checklist
npm testpasses (all tests green)npx tsc --noEmitis clean (no type errors).js, noany, noconsole.log)Test plan
Related issues