Skip to content

Add real VS Code integration smoke test - #144

Open
rcosta358 wants to merge 7 commits into
codex/issue-130-test-apifrom
codex/issue-131-vscode-smoke-tests
Open

rcosta358 wants to merge 7 commits into
codex/issue-130-test-apifrom
codex/issue-131-vscode-smoke-tests

Conversation

@rcosta358

@rcosta358 rcosta358 commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Adds a real VS Code stable smoke test for activation, readiness, and the refinement diagnostic returned after Verify in an isolated failing workspace.

CI runs it on pull requests and main with Java 21 and xvfb, and uploads VS Code logs on failure.

Validated: smoke test locally and in CI, npm ci, lint, production/test type checks, and extension installation.

Depends on #143.

Closes #131.

Generated by Codex.

@rcosta358 rcosta358 added the testing Testing related label Oct 2, 2026
rcosta358 and others added 3 commits October 6, 2026 23:22
# Conflicts:
#	.github/workflows/test.yml
#	client/.vscodeignore
#	client/package-lock.json
#	client/package.json
#	client/tsconfig.json

Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
Co-authored-by: Codex <noreply@openai.com>
@rcosta358

Copy link
Copy Markdown
Collaborator Author

While restacking the prerequisites for #147, the new PR/main trigger policy exposed the old same-repository PR skip on this smoke-test job. Removed that condition in 12db357 and propagated the update through the stack. The final CI run passed both Checks and the real VS Code integration test; independent review and extension installation also passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Testing related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants