Skip to content

Conversation

@d-klotz
Copy link
Contributor

@d-klotz d-klotz commented Dec 15, 2025

Summary

  • Add test/ to the files array in devtools package.json
  • Add PR guidelines documentation to CLAUDE.md

Problem

The devtools/index.js exports the test module:

const test = require('./test');
module.exports = { createFriggInfrastructure, ...test };

But the test/ directory was not included in the npm package. This caused builds to fail with:

Error: Cannot find module './test'
Require stack:
- .../node_modules/@friggframework/devtools/index.js

Solution

Add test/ to the files array in packages/devtools/package.json to ensure it's published to npm.

Test plan

  • Verified npm run frigg:local:build works after fix
  • Publish new version of @friggframework/devtools
  • Install new version in a project
  • Verify frigg build works without module not found errors
📦 Published PR as canary version: 2.0.0--canary.520.6641121.0

✨ Test out this PR locally via:

npm install @friggframework/core@2.0.0--canary.520.6641121.0
npm install @friggframework/devtools@2.0.0--canary.520.6641121.0
npm install @friggframework/eslint-config@2.0.0--canary.520.6641121.0
npm install @friggframework/prettier-config@2.0.0--canary.520.6641121.0
npm install @friggframework/schemas@2.0.0--canary.520.6641121.0
npm install @friggframework/serverless-plugin@2.0.0--canary.520.6641121.0
npm install @friggframework/test@2.0.0--canary.520.6641121.0
npm install @friggframework/ui@2.0.0--canary.520.6641121.0
# or 
yarn add @friggframework/core@2.0.0--canary.520.6641121.0
yarn add @friggframework/devtools@2.0.0--canary.520.6641121.0
yarn add @friggframework/eslint-config@2.0.0--canary.520.6641121.0
yarn add @friggframework/prettier-config@2.0.0--canary.520.6641121.0
yarn add @friggframework/schemas@2.0.0--canary.520.6641121.0
yarn add @friggframework/serverless-plugin@2.0.0--canary.520.6641121.0
yarn add @friggframework/test@2.0.0--canary.520.6641121.0
yarn add @friggframework/ui@2.0.0--canary.520.6641121.0

Version

Published prerelease version: v2.0.0-next.64

Changelog

🐛 Bug Fix

  • @friggframework/devtools
    • fix(devtools): include test/ in npm package files #520 (@d-klotz)
  • @friggframework/core, @friggframework/devtools, @friggframework/eslint-config, @friggframework/prettier-config, @friggframework/schemas, @friggframework/serverless-plugin, @friggframework/test, @friggframework/ui
    • fix(devtools): include infrastructure/ in npm package files #519 (@d-klotz)

Authors: 1

The devtools/index.js exports the test module, but the test/ directory
was not included in the package.json files array. This caused builds
to fail with:

Error: Cannot find module './test'

Add test/ to the files array to ensure it's published to npm.
Add documentation about creating pull requests for the Frigg Framework:
- Always target the 'next' branch
- Always add 'release' and 'prerelease' labels
- Ensure project compiles before creating PR
@d-klotz d-klotz added release Create a release when this pr is merged prerelease This change is available in a prerelease. labels Dec 15, 2025
@sonarqubecloud
Copy link

@d-klotz d-klotz merged commit 1133f45 into next Dec 15, 2025
7 of 8 checks passed
@d-klotz d-klotz deleted the fix/include-test-in-devtools-npm-package branch December 15, 2025 18:57
@seanspeaks
Copy link
Contributor

🚀 PR was released in v2.0.0-next.67 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

prerelease This change is available in a prerelease. release Create a release when this pr is merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants