Skip to content

chore: fix npmrc dev dependency config#428

Merged
joryirving merged 1 commit into
mainfrom
chore/fix-npmrc-omit
Jun 17, 2026
Merged

chore: fix npmrc dev dependency config#428
joryirving merged 1 commit into
mainfrom
chore/fix-npmrc-omit

Conversation

@joryirving

Copy link
Copy Markdown
Contributor

Closes #411

@joryirving joryirving merged commit d6d4fb5 into main Jun 17, 2026
3 checks passed
@joryirving joryirving deleted the chore/fix-npmrc-omit branch June 17, 2026 16:41

@its-saffron its-saffron Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

AI Automated Review

Full PR review.

Analysis engine: review@https://litellm.jory.dev/v1 (openai) — primary route

Summary

This PR fixes the .npmrc configuration to ensure that devDependencies are correctly included during npm ci operations. This change directly addresses the recommendation in issue PR 411 to replace the omit= line with include=dev to ensure a reproducible local validation environment where npm run typecheck and npm run test can function correctly from a clean checkout.

Change-by-change findings

  • .npmrc: Replaced omit= with include=dev and updated the descriptive comment to reflect the correct configuration syntax.

Linked Issue Fit

  • Issue PR 411: The PR implements the exact recommendation provided in the issue: "Replace the .npmrc omit= line with include=dev (or delete the file)".

Standards Compliance

  • The change follows the technical debt remediation process as outlined in the linked issue.

Must Checks

  • Check for breaking API changes in updated dependencies: N/A. No dependencies were upgraded in this PR; this is a configuration change affecting how dependencies are installed.
  • Run full test suite after upgrade: The change affects the installation of devDependencies (required for testing). CI reports that Validate and Docker Build were successful, indicating that the change does not break the build or the test environment.

Unknowns or Needs Verification

  • None.

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.

Replace the .npmrc omit= line with include=dev (or delete the file) and verify CI is still clean

1 participant