4 Commits

Author	SHA1	Message	Date
hurui200320	e2b127b7e5	fix(e2e): replace naive OpenAI key-presence check with live API probe in E2E suite setups ... The existing actor-selection logic in several E2E suite setups checked only whether OPENAI_API_KEY was present (non-empty). A valid key that has hit its quota limit passes that check but fails at runtime with HTTP 429, causing the test to fail even though Anthropic credits are available. Changes: - Add robot/e2e/check_openai_key.py: stdlib-only (urllib.request) script that sends a minimal chat-completion request ('Hi', max_tokens=1, gpt-4o-mini) to the OpenAI API. Exits 0 on HTTP 200; exits 1 for quota (429), auth (401), network errors, or any other failure. - Add 'Resolve LLM Actor' keyword to robot/e2e/common_e2e.resource: runs the probe script via ${PYTHON} and returns the openai_model argument (default openai/gpt-4o) on success, or the anthropic_model argument (default anthropic/claude-sonnet-4-20250514) on failure. Skips the probe entirely when OPENAI_API_KEY is not set. - Update m6_acceptance.robot, wf04_multi_project.robot, wf05_db_migration.robot, wf07_cicd.robot, and wf16_devcontainer.robot to use 'Resolve LLM Actor' instead of the inline has_openai boolean check. No production source code (src/) is modified. The decision to fall back to Anthropic is made once per suite setup, before any test runs. Closes #10198	2026-04-17 18:00:47 +08:00
freemo	8ea00f5185	fix: restore CI quality tests to passing state (#4175) ... Co-authored-by: Jeffrey Phillips Freeman <the@jeffreyfreeman.me> Co-committed-by: Jeffrey Phillips Freeman <the@jeffreyfreeman.me>	2026-04-08 11:02:14 +00:00
freemo	48cff5cfe0	refactor(cli): rename plan lifecycle-list and lifecycle-apply to match specification ... Renames `plan lifecycle-list` to `plan list` and `plan lifecycle-apply` to `plan apply` to align with the specification's canonical command names. Removes legacy V2 plan commands that occupied those names. - Renamed CLI command registrations from lifecycle-list/lifecycle-apply to list/apply - Removed legacy V2 apply and list commands (~200 lines) - Updated apply shortcut in main.py to delegate to v3 lifecycle - Added defensive null check for plan existence in apply command - Updated 63+ test, doc, and benchmark files for consistency Closes #881 Co-authored-by: Jeffrey Phillips Freeman <the@jeffreyfreeman.me> Co-committed-by: Jeffrey Phillips Freeman <the@jeffreyfreeman.me>	2026-04-02 19:09:04 +00:00
hurui200320	b6c3169634	test(e2e): workflow example 16 — devcontainer-driven development (supervised profile) ... Implements E2E test for devcontainer-driven development workflow. Tests project setup with .devcontainer config, resource registration, and full plan lifecycle with supervised automation profile. Changes from Cycle 5 review: 1. Replaced custom WF16_ALLOW_XFAIL_SKIP escape hatch with the project standard tdd_expected_fail tag system. Test is now tagged [Tags] tdd_expected_fail tdd_bug tdd_bug_762 and the listener inverts failures in CI until devcontainer features are fully wired. 2. Added diagnostic WF16 Test Teardown keyword that captures plan status JSON on failure, mirroring the WF05 teardown pattern. 3. Added stderr checks to all Traceback/INTERNAL guard assertions so tracebacks appearing only in stderr are also detected. 4. Rebased branch onto latest master (e4c01492). 5. Documented timeout budget: added comment explaining theoretical worst-case (~46min) vs realistic execution under the 35min limit. 6. Added fail-fast warning in AC-3 check noting that AC-4, AC-5, and AC-6 are dependent on AC-3 and are likely to also fail. 7. Reordered plan apply arguments to --yes --format json <plan_id> for spec consistency. 8. Decomposed long ready_for_apply Evaluate expression into three intermediate booleans (is_apply_phase, is_execute_complete, is_already_applied) for readability. 9. Added shared_session.close() calls in resource_dag.robot before each final print() to ensure explicit session cleanup. 10. Cleaned up stale xfail/skip classification references in log messages. Changes from earlier cycles (3-4) retained in this commit: - Terminal state validation, AC output combination, JSON format flags, suite-level init, Force Tags, automation profile fallback, ULID regex IGNORECASE, AC-5 Evaluate decomposition, defensive re-check pattern. ISSUES CLOSED: #762	2026-03-30 18:45:10 +08:00