fix(cli): typecheck + lint after rebase on cli-subprocess

- spawnChild is now sync (matches cli-subprocess); update WatchOpts type
  and watch loop accordingly.
- Bail with handleCancel when no env can be resolved (was passing
  string | undefined into typed APIs).
- Add explicit return type on the keyring factory closure.
- Auto-fixed quote style in secret-refs.

Author: HugoRCD

packages/cli/src/commands/run.ts

@@ -98,6 +98,7 @@ export default defineCommand({
     let projectData: Project | null = null
     let environment: Environment | null = null
     const envName = runArgs.env || defaultEnv
+    if (!envName) handleCancel('No environment specified. Pass --env or set `defaultEnv` in shelve.json.')
     const cacheInput = (env: string): CacheKeyInput => ({
       url,
       teamSlug: slug,
@@ -297,7 +298,7 @@ type WatchOpts = {
   template: ParsedTemplate | null
   restartOnChange: boolean
   getChild: () => ChildProcess
-  spawnNew: (env: NodeJS.ProcessEnv) => Promise<ChildProcess>
+  spawnNew: (env: NodeJS.ProcessEnv) => ChildProcess
 }
 
 function fingerprint(variables: EnvVarExport[]): string {
@@ -330,11 +331,13 @@ function startWatch(opts: WatchOpts): void {
         if (opts.restartOnChange) {
           const child = opts.getChild()
           if (child.pid && process.platform !== 'win32') {
-            try { process.kill(-child.pid, 'SIGTERM') } catch { /* ignore */ }
+            try {
+              process.kill(-child.pid, 'SIGTERM') 
+            } catch { /* ignore */ }
           } else {
             child.kill('SIGTERM')
           }
-          await opts.spawnNew(env)
+          opts.spawnNew(env)
         } else {
           const child = opts.getChild()
           try {

packages/cli/src/services/credentials.ts

@@ -30,7 +30,7 @@ async function loadKeyring(): Promise<((service: string, account: string) => Key
 
   try {
     const mod = await import('@napi-rs/keyring')
-    keyringFactory = (service, account) => new mod.Entry(service, account) as unknown as KeyringEntry
+    keyringFactory = (service: string, account: string): KeyringEntry => new mod.Entry(service, account) as unknown as KeyringEntry
   } catch (err) {
     if (DEBUG) consola.warn(`Keyring unavailable, falling back to file storage: ${err}`)
     keyringFactory = null

packages/cli/src/utils/ignore-files.ts

@@ -44,7 +44,7 @@ function upsertBlock(existing: string): string {
     return existing + sep + (existing.length === 0 ? '' : '\n') + block
   }
   const footerIdx = existing.indexOf(FOOTER_TAG, headerIdx)
-  if (footerIdx === -1) return existing + '\n' + block
+  if (footerIdx === -1) return `${existing }\n${ block}`
   const before = existing.slice(0, headerIdx)
   const after = existing.slice(footerIdx + FOOTER_TAG.length).replace(/^\n/, '')
   return before + block + after

packages/cli/src/utils/secret-refs.ts

@@ -38,7 +38,7 @@ export function parseEnvTemplate(content: string): ParsedTemplate {
     let value = line.slice(eqIdx + 1).trim()
     if (
       (value.startsWith('"') && value.endsWith('"'))
-      || (value.startsWith("'") && value.endsWith("'"))
+      || (value.startsWith('\'') && value.endsWith('\''))
     ) {
       value = value.slice(1, -1)
     }