Skip to content

fix: detect package manager from lockfile in single-package repos#548

Merged
nam-hle merged 2 commits intomainfrom
test/single-package-repo
Feb 24, 2026
Merged

fix: detect package manager from lockfile in single-package repos#548
nam-hle merged 2 commits intomainfrom
test/single-package-repo

Conversation

@nam-hle
Copy link
Copy Markdown
Member

@nam-hle nam-hle commented Feb 24, 2026

Summary

  • Fixes a bug where single-package (non-monorepo) repos always detected packageManager: "npm" regardless of the lockfile present
  • Adds detectPackageManager() in @nadle/project-resolver that checks for pnpm-lock.yaml, yarn.lock, and package-lock.json
  • Adds integration tests for single-package npm, pnpm, and yarn repos (previously only monorepo scenarios were tested)

Test plan

  • pnpm -F nadle test project-dir — all 7 tests pass (4 existing monorepo + 3 new single-package)
  • pnpm -F @nadle/project-resolver test — all 34 tests pass
  • CI passes on Ubuntu, macOS, and Windows

🤖 Generated with Claude Code

nam-hle and others added 2 commits February 24, 2026 22:55
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 24, 2026

Deploy Preview for nadle ready!

Name Link
🔨 Latest commit c73af2c
🔍 Latest deploy log https://app.netlify.com/projects/nadle/deploys/699dcd05446659000737dad9
😎 Deploy Preview https://deploy-preview-548--nadle.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 94
Accessibility: 98
Best Practices: 100
SEO: 100
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@nam-hle nam-hle enabled auto-merge (squash) February 24, 2026 16:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

size-limit report 📦

Path Size
bundled 124.48 KB (0%)

@nam-hle nam-hle disabled auto-merge February 24, 2026 16:10
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Feb 24, 2026

Open in StackBlitz

create-nadle

npx https://pkg.pr.new/nadlejs/nadle/create-nadle@548

eslint-plugin-nadle

npx https://pkg.pr.new/nadlejs/nadle/eslint-plugin-nadle@548

@nadle/kernel

npx https://pkg.pr.new/nadlejs/nadle/@nadle/kernel@548

@nadle/language-server

npx https://pkg.pr.new/nadlejs/nadle/@nadle/language-server@548

nadle

npx https://pkg.pr.new/nadlejs/nadle@548

@nadle/project-resolver

npx https://pkg.pr.new/nadlejs/nadle/@nadle/project-resolver@548

nadle-vscode-extension

npx https://pkg.pr.new/nadlejs/nadle/nadle-vscode-extension@548

commit: c73af2c

@nam-hle nam-hle merged commit 0a3bf36 into main Feb 24, 2026
16 of 20 checks passed
@nam-hle nam-hle deleted the test/single-package-repo branch February 24, 2026 16:10
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