Skip to content

fix: report invalid environment JSON#944

Open
I-am-Uchenna wants to merge 1 commit into
AnWeber:mainfrom
I-am-Uchenna:agent/report-invalid-environment-json
Open

fix: report invalid environment JSON#944
I-am-Uchenna wants to merge 1 commit into
AnWeber:mainfrom
I-am-Uchenna:agent/report-invalid-environment-json

Conversation

@I-am-Uchenna

Copy link
Copy Markdown

Closes #939

Summary

  • keep missing IntelliJ environment files optional
  • stop treating malformed existing environment files as missing
  • report the failing file and original JSON parser message
  • cover both http-client.env.json and http-client.private.env.json

Testing

  • node --experimental-vm-modules node_modules/jest/bin/jest.js --runInBand --runTestsByPath src/plugins/intellij/intellijVariableProvider.spec.ts
  • npm run eslint
  • npm run tsc:check
  • npm run build
  • npm run lockfile-lint
  • npm run packageJson-lint
  • npx prettier --check src/plugins/intellij/intellijVariableProvider.ts src/plugins/intellij/intellijVariableProvider.spec.ts

The full suite was also run locally: 46 of 56 suites passed. The remaining 10 failures are in unrelated Windows path and CRLF-sensitive fixture tests; the focused IntelliJ suite passes all 6 tests.

@I-am-Uchenna
I-am-Uchenna marked this pull request as ready for review July 23, 2026 21:03
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.

Misleading/confusing errors when http-client.env.json fails to parse

1 participant