Skip to content

chore(CI/CD): Prevent agents from running npm#8930

Merged
lstein merged 5 commits intomainfrom
lstein/cicd/prevent-agents-from-running-npm
Apr 27, 2026
Merged

chore(CI/CD): Prevent agents from running npm#8930
lstein merged 5 commits intomainfrom
lstein/cicd/prevent-agents-from-running-npm

Conversation

@lstein
Copy link
Copy Markdown
Collaborator

@lstein lstein commented Feb 28, 2026

Summary

Copilot, and presumably other coding agents, have a tendency to run npm when updating the front end code. This creates an unwanted package-lock.json file.

This PR does three things:

  1. It adds an AGENTS.md file that gives coding agents explicit instructions to use pnpm rather than npm.
  2. It adds a packageManager instruction to invokeai/frontend/web/scripts/package.json (which seemed to be missing)
  3. It adds a check to the frontendchecks.yml workflow that will generate an error if package-lock.json is found in the PR.

Related Issues / Discussions

QA Instructions

  1. Using a coding agent, ask it to perform a pnpm operation such as lint or build.
  2. Confirm that it uses the proper tool.
  3. Add a package-lock.json file to invokeai/frontend/web and try to push it into this PR. The frontend-checks check should fail.

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@lstein lstein self-assigned this Feb 28, 2026
@github-actions github-actions Bot added CI-CD Continuous integration / Continuous delivery frontend-deps PRs that change frontend dependencies frontend PRs that change frontend files labels Feb 28, 2026
@lstein lstein assigned Pfannkuchensack and unassigned lstein Feb 28, 2026
@lstein lstein changed the title chore(CI/CD): Prevent agents from runing npm chore(CI/CD): Prevent agents from running npm Feb 28, 2026
@lstein lstein added v6.12.0 Intended for 6.12.0 release v6.13.x and removed v6.12.0 Intended for 6.12.0 release labels Mar 10, 2026
@lstein lstein merged commit 62d21a5 into main Apr 27, 2026
16 checks passed
@lstein lstein deleted the lstein/cicd/prevent-agents-from-running-npm branch April 27, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-CD Continuous integration / Continuous delivery frontend PRs that change frontend files frontend-deps PRs that change frontend dependencies v6.13.x

Projects

Status: 6.13.x Theme: MODELS

Development

Successfully merging this pull request may close these issues.

2 participants