fix: no longer includes tag in sanity devDependency range

Author: snorrees

src/actions/init.ts

@@ -1,13 +1,10 @@
 import path from 'path'
 import {inject} from './inject'
 import {ensureDir, writeFile} from '../util/files'
-import {resolveLatestVersions} from '../npm/resolveLatestVersions'
 import sharedFlags from '../sharedFlags'
 import {TypedFlags} from 'meow'
 import {getPackage} from '../npm/package'
 import {defaultSourceJs, defaultSourceTs} from '../configs/default-source'
-import {incompatiblePluginPackage} from '../constants'
-import {forcedPackageVersions} from '../configs/forced-package-versions'
 
 export const initFlags = {
   ...sharedFlags,

src/configs/forced-package-versions.ts

@@ -1,8 +1,16 @@
+const sanityVersion = '3.0.0-rc.0'
+
 export const forcedPackageVersions = {
   react: '^18',
   '@types/react': '^18',
   'react-dom': '^18',
   '@types/react-dom': '^18',
-  sanity: 'dev-preview || 3.0.0-rc.0',
+  sanity: `dev-preview || ${sanityVersion}`,
   '@sanity/ui': '1.0.0-beta.32',
-} as const
+}
+
+export const forcedDevPackageVersions = {
+  ...forcedPackageVersions,
+  // tagged release does not play nice with npm ci
+  sanity: sanityVersion,
+}

src/npm/package.ts

@@ -13,7 +13,7 @@ import {cliName, incompatiblePluginPackage} from '../constants'
 import {InjectOptions, PackageData} from '../actions/inject'
 import {expectedScripts} from '../actions/verify/validations'
 import {PackageJson} from '../actions/verify/types'
-import {forcedPackageVersions} from '../configs/forced-package-versions'
+import {forcedDevPackageVersions, forcedPackageVersions} from '../configs/forced-package-versions'
 
 const defaultDependencies = [incompatiblePluginPackage]
 
@@ -234,11 +234,14 @@ export async function writePackageJson(data: PackageData, options: InjectOptions
     ...(addDeps || {}),
     ...(await resolveLatestVersions(defaultDependencies)),
   })
-  const devDependencies = forceDependencyVersions({
-    ...(addDevDeps || {}),
-    ...(prev.devDependencies || {}),
-    ...(await resolveLatestVersions([...newDevDependencies, ...defaultDevDependencies])),
-  })
+  const devDependencies = forceDependencyVersions(
+    {
+      ...(addDevDeps || {}),
+      ...(prev.devDependencies || {}),
+      ...(await resolveLatestVersions([...newDevDependencies, ...defaultDevDependencies])),
+    },
+    forcedDevPackageVersions
+  )
   const peerDependencies = forceDependencyVersions({
     ...(prev.peerDependencies || {}),
     ...(addPeers || {}),
@@ -384,10 +387,13 @@ export function sortKeys<T extends Record<string, unknown>>(unordered: T): T {
     }, {} as T)
 }
 
-function forceDependencyVersions(deps: Record<string, string>): Record<string, string> {
+function forceDependencyVersions(
+  deps: Record<string, string>,
+  versions = forcedPackageVersions
+): Record<string, string> {
   const entries = Object.entries(deps).map((entry) => {
     const [pkg] = entry
-    const forceVersion = forcedPackageVersions[pkg as keyof typeof forcedPackageVersions]
+    const forceVersion = versions[pkg as keyof typeof versions]
     if (forceVersion) {
       return [pkg, forceVersion]
     }