Skip to content

Fix Node 22 SPFx sample deploy failure by correcting ESLint plugin runtime dependency#16075

Draft
Copilot wants to merge 2 commits into
devfrom
copilot/fix-sample-todo-list-github-action
Draft

Fix Node 22 SPFx sample deploy failure by correcting ESLint plugin runtime dependency#16075
Copilot wants to merge 2 commits into
devfrom
copilot/fix-sample-todo-list-github-action

Conversation

Copilot AI commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

The sample-todo-list-with-spfx|ubuntu-latest|node 22|dev Actions job failed during atk deploy --env dev because ESLint could not load typescript-eslint from the shared flat config. This PR fixes the dependency declaration so the config resolves correctly in CI/runtime install contexts.

  • Root cause in shared lint config packaging

    • packages/eslint-plugin-teamsfx/config/shared.mjs imports typescript-eslint at runtime.
    • typescript-eslint was declared only in devDependencies, which is insufficient when the config is consumed outside that package’s dev environment.
  • Dependency declaration fix

    • Moved typescript-eslint from devDependencies to dependencies in packages/eslint-plugin-teamsfx/package.json.
    • Updated packages/eslint-plugin-teamsfx/pnpm-lock.yaml to match the new dependency scope.
  • Impact

    • Ensures consumers of @microsoft/eslint-plugin-teamsfx can resolve typescript-eslint whenever config/shared.mjs is imported (including SPFx sample deploy/lint in CI on Node 22).
{
  "dependencies": {
    "requireindex": "^1.2.0",
    "typescript-eslint": "^8.56.0"
  }
}

Copilot AI changed the title [WIP] Fix failing GitHub Actions job for sample todo list Fix Node 22 SPFx sample deploy failure by correcting ESLint plugin runtime dependency Jun 2, 2026
Copilot AI requested a review from neil-yechenwei June 2, 2026 08:13
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.

2 participants