Skip to content

chore: update dependencies and improve code consistency - #3338

Open
Dmitrii Shilov (bk201-) wants to merge 7 commits into
mainfrom
dev/dshilov/bump-deps-2026-08
Open

chore: update dependencies and improve code consistency#3338
Dmitrii Shilov (bk201-) wants to merge 7 commits into
mainfrom
dev/dshilov/bump-deps-2026-08

Conversation

@bk201-

@bk201- Dmitrii Shilov (bk201-) commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

This pull request makes extensive updates to the project's linting setup by removing ESLint in favor of a custom Oxlint plugin, updating dependencies, and refining VS Code configurations. The main goals are to streamline linting, enforce consistent code quality rules, and keep dependencies up to date. Additionally, it introduces minor improvements and bug fixes in language service error handling.

Linting and Tooling Overhaul

  • Removed ESLint configuration and related dependencies, fully migrating linting to Oxlint with a new custom plugin (.oxlint-plugin.mjs). This plugin enforces license headers, import restrictions, and l10n usage conventions, and is integrated via .oxlintrc.jsonc. (eslint.config.mjs removed, .oxlintrc.jsonc, .oxlint-plugin.mjs, package.json, .vscode/settings.json, .vscode/tasks.json, .vscode/extensions.json) [1] [2] [3] [4] [5] [6] [7] [8]
  • Updated VS Code settings to use Oxlint for code actions on save and removed the TypeScript SDK override, reflecting the shift away from ESLint. (.vscode/settings.json) [1] [2]
  • Modified the lint task to remove ESLint-specific problem matchers. (.vscode/tasks.json)

Dependency and Engine Updates

  • Upgraded Node.js and VS Code engine requirements to the latest versions, and updated a wide range of dependencies and devDependencies for compatibility and improved stability. (.nvmrc, package.json, packages/nosql-language-service/package.json) [1] [2] [3] [4] [5]

Custom Oxlint Plugin Implementation

  • Added .oxlint-plugin.mjs with rules for:
    • Enforcing a standard Microsoft license header,
    • Restricting certain imports to encourage isomorphic code,
    • Enforcing direct usage of @vscode/l10n instead of vscode.l10n. (.oxlint-plugin.mjs)
  • Updated .oxlintrc.jsonc to use the new plugin and rules, removing the old ESLint plugin and configuration. (.oxlintrc.jsonc) [1] [2]

Other Improvements and Fixes

  • Added a new localization string for "Waiting for Azure Cosmos DB permissions to propagate…". (l10n/bundle.l10n.json)
  • Improved error recovery in the SQL language service to handle cases where EOF token locations are unavailable. (packages/nosql-language-service/src/errors/SqlError.test.ts)

These changes modernize the development workflow, ensure code quality through more robust linting, and keep the project up to date with the latest dependencies and tooling.

@bk201-
Dmitrii Shilov (bk201-) requested a review from a team as a code owner August 31, 2026 15:04
@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

✅ Build (Compile, Lint, Prettier, l10n, Package)

🧱 Step Results

  • Validate Version: ✅ success
  • Localization: ✅ success
  • Lint: ✅ success
  • Prettier: ✅ success
  • Compile (tsc): ✅ success
  • Package: ✅ success
  • External Skills (info): ✅ success

📥 Artifacts (run)

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

🔬 NoSQL language-service integration

🧪 Result

  • NoSQL integration tests: ✅ success

📥 Artifacts (run)

@github-code-quality

github-code-quality Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Code Coverage Overview

Languages: TypeScript

TypeScript / code-coverage/vitest

The overall line coverage in commit 4955391 in the dev/dshilov/bump-dep... branch remains at 77%, unchanged from commit 8a1e1b4 in the main branch.

Show a line coverage summary of the most impacted files.
File main 8a1e1b4 dev/dshilov/bump-dep... 4955391 +/-
src/cosmosdb/se...umentSession.ts 21% 21% 0%
packages/nosql-...ce/src/index.ts 94% 94% 0%
src/panels/acco...ries/collect.ts 2% 2% 0%
packages/nosql-...lexer/tokens.ts 100% 100% 0%
src/constants.ts 46% 48% +2%
src/utils/azureClients.ts 0% 33% +33%
src/cosmosdb/ut...ls/rbacUtils.ts 11% 51% +40%

Updated August 31, 2026 15:19 UTC

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

🧪 Tests (Unit + Integration)

🧪 Results

  • Unit Tests (vitest): ✅ success
  • Integration Tests (extension host): ✅ success

📈 Coverage

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

🎭 E2E Tests (Playwright + VS Code)

🧪 Result

  • E2E Tests: ✅ success

📥 Artifacts (run)

Tip: the HTML report artifact contains a self-contained Playwright report.
Download the zip, extract, and open index.html — or run
npx playwright show-report <extracted-dir> for the interactive view.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bk201- Dmitrii Shilov (bk201-) changed the title dev/dshilov/bump deps 2026 08 chore: update dependencies and improve code consistency Sep 1, 2026
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.

1 participant