Skip to content

Improve e2e tests to check for a "ready" event emitted by the UI#117

Merged
jviotti merged 1 commit intomainfrom
fix-broken-cli-path
Dec 17, 2025
Merged

Improve e2e tests to check for a "ready" event emitted by the UI#117
jviotti merged 1 commit intomainfrom
fix-broken-cli-path

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Dec 17, 2025

Fixes: #116
Signed-off-by: Juan Cruz Viotti jv@jviotti.com

@jviotti jviotti force-pushed the fix-broken-cli-path branch 2 times, most recently from 43c3d6a to a75cf0e Compare December 17, 2025 12:06
@augmentcode
Copy link

augmentcode bot commented Dec 17, 2025

🤖 Augment PR Summary

Summary: Fixes the VS Code extension’s bundled CLI entrypoint so lint/format/metaschema commands run correctly (addresses #116).

Changes:

  • Update `CommandExecutor` to target `@sourcemeta/jsonschema/npm/cli.js`
  • Add a new VS Code e2e test that asserts lint diagnostics are produced for a known-bad schema
  • Update the HTTP `$ref` fixture schema to include a `description` so it remains lint-clean
  • Improve the webview tabs to better surface “command failed” states and display raw CLI output

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 2 suggestions posted. Focused on test robustness and user-visible error reporting.

Comment augment review to trigger a new review at any time.

@jviotti jviotti force-pushed the fix-broken-cli-path branch 3 times, most recently from 8eb9278 to 85a4f9d Compare December 17, 2025 12:26
@jviotti jviotti changed the title Fix incorrect CLI path in the extension Improve e2e tests to check for a "ready" event emitted by the UI Dec 17, 2025
Also extend the extension and the e2e test suite to better catch and
report these issues that can lead to a non-working UI despite of a
working extension in the background (which made the existing e2e suite
pass).

Fixes: #116
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti force-pushed the fix-broken-cli-path branch from 85a4f9d to aa87eb1 Compare December 17, 2025 12:28
@jviotti jviotti merged commit 745d682 into main Dec 17, 2025
4 checks passed
@jviotti jviotti deleted the fix-broken-cli-path branch December 17, 2025 12:31
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.

v0.0.7 is broken

1 participant