fix(intent): read local package.json version before npm registry fallback (#104)

Author: RazorAlexMachin

.changeset/fix-local-version-fallback.md

@@ -0,0 +1,5 @@
+---
+'@tanstack/intent': patch
+---
+
+Read local package.json version before falling back to npm registry in `intent stale`. This fixes version drift detection for packages not published to public registry.npmjs.org (e.g. GitHub Packages, Artifactory, private registries).

packages/intent/src/staleness.ts

@@ -34,9 +34,20 @@ function classifyVersionDrift(
 }
 
 // ---------------------------------------------------------------------------
-// npm version fetching
+// Version resolution
 // ---------------------------------------------------------------------------
 
+function readLocalVersion(packageDir: string): string | null {
+  try {
+    const pkgJson = JSON.parse(
+      readFileSync(join(packageDir, 'package.json'), 'utf8'),
+    ) as Record<string, unknown>
+    return typeof pkgJson.version === 'string' ? pkgJson.version : null
+  } catch {
+    return null
+  }
+}
+
 async function fetchNpmVersion(packageName: string): Promise<string | null> {
   try {
     const res = await fetch(
@@ -50,6 +61,13 @@ async function fetchNpmVersion(packageName: string): Promise<string | null> {
   }
 }
 
+async function fetchCurrentVersion(
+  packageDir: string,
+  packageName: string,
+): Promise<string | null> {
+  return readLocalVersion(packageDir) ?? (await fetchNpmVersion(packageName))
+}
+
 // ---------------------------------------------------------------------------
 // Sync state
 // ---------------------------------------------------------------------------
@@ -141,8 +159,8 @@ export async function checkStaleness(
   const skillVersion =
     skillMetas.find((s) => s.libraryVersion)?.libraryVersion ?? null
 
-  // Fetch current npm version
-  const currentVersion = await fetchNpmVersion(library)
+  // Resolve current version: prefer local package.json, fall back to npm registry
+  const currentVersion = await fetchCurrentVersion(packageDir, library)
 
   // Classify drift
   const versionDrift =

packages/intent/tests/staleness.test.ts

@@ -309,4 +309,47 @@ describe('checkStaleness', () => {
     expect(report.skills).toHaveLength(1)
     expect(requireFirstSkill(report).needsReview).toBe(false)
   })
+
+  it('reads version from local package.json when npm fetch fails', async () => {
+    writeFileSync(
+      join(tmpDir, 'package.json'),
+      JSON.stringify({ name: '@private/lib', version: '2.5.0' }),
+    )
+
+    writeSkill(tmpDir, 'core', {
+      name: 'core',
+      description: 'Core',
+      library_version: '2.0.0',
+    })
+
+    mockFetchNotOk()
+
+    const report = await checkStaleness(tmpDir, '@private/lib')
+    expect(report.currentVersion).toBe('2.5.0')
+    expect(report.versionDrift).toBe('minor')
+    const skill = requireFirstSkill(report)
+    expect(skill.needsReview).toBe(true)
+    expect(skill.reasons[0]).toContain('version drift')
+  })
+
+  it('prefers local package.json over npm registry', async () => {
+    writeFileSync(
+      join(tmpDir, 'package.json'),
+      JSON.stringify({ name: '@example/lib', version: '3.0.0' }),
+    )
+
+    writeSkill(tmpDir, 'core', {
+      name: 'core',
+      description: 'Core',
+      library_version: '2.0.0',
+    })
+
+    // npm returns an older published version
+    mockFetchVersion('2.5.0')
+
+    const report = await checkStaleness(tmpDir, '@example/lib')
+    // Local package.json should take precedence
+    expect(report.currentVersion).toBe('3.0.0')
+    expect(report.versionDrift).toBe('major')
+  })
 })