Skip to content

Add build warnings for missing node_modules and update NuGet tags#33

Merged
antosubash merged 1 commit intomainfrom
claude/modest-gould
Mar 30, 2026
Merged

Add build warnings for missing node_modules and update NuGet tags#33
antosubash merged 1 commit intomainfrom
claude/modest-gould

Conversation

@antosubash
Copy link
Copy Markdown
Owner

Changes

  • NuGet Tags: Added simplemodule tag to Directory.Build.props for better package discoverability
  • Build Warnings: Added ViteBuildCheck target to SimpleModule.Hosting.targets to warn when node_modules is missing during Vite builds
  • Module Build Warnings: Added JsBuildCheck target to modules/Directory.Build.targets for missing node_modules in module JavaScript builds
  • Auto-generated Types: Updated TypeScript type definitions in Admin, OpenIddict, and Users modules with new DTO properties and interfaces
  • Dependencies: Updated package-lock.json with peer dependency markings for various packages

- Add ViteBuildCheck and JsBuildCheck MSBuild targets that emit a build
  warning when package.json exists but node_modules is not found, so
  developers know frontend JS won't be compiled on a fresh clone/worktree
- Add 'simplemodule' to PackageTags so published NuGet packages appear
  in the Marketplace module's NuGet search
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying simplemodule-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: a9d2ab3
Status: ✅  Deploy successful!
Preview URL: https://6eb92385.simplemodule-website.pages.dev
Branch Preview URL: https://claude-modest-gould.simplemodule-website.pages.dev

View logs

@antosubash antosubash merged commit b302d7c into main Mar 30, 2026
3 checks passed
@antosubash antosubash deleted the claude/modest-gould branch April 1, 2026 15:15
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