itiger 0.1.0 → 0.1.1

package/README.md

@@ -18,8 +18,10 @@ When installing an `@tiger/*` package, `itiger` adds
 explicit `--@tiger:registry` option is already present. Non-Tiger packages are
 delegated without registry changes.
 
-Requirements: Node.js 18 or newer, plus the local `skills` CLI for `skills`
-commands and npm for package installation.
+Requirements: Node.js 18 or newer, plus npm for package installation. For
+`skills` commands, the locally installed `skills` CLI is used when available;
+otherwise `npx --yes skills` downloads and runs the public CLI without
+persisting credentials or changing the user's global installation.
 
 ## Development
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "itiger",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "A thin adapter for internal skills and @tiger npm packages.",
   "type": "commonjs",
   "bin": {

package/src/process.js

@@ -1,21 +1,34 @@
 const { spawn } = require('node:child_process');
 
-function runCommand(command, args) {
+function runCommand(command, args, fallback) {
   return new Promise((resolve) => {
     let child;
+    let settled = false;
     try {
       child = spawn(command, args, { stdio: 'inherit' });
     } catch (error) {
-      process.stderr.write(`itiger: unable to start ${command}: ${error.message}\n`);
-      resolve(1);
+      if (fallback) {
+        runCommand(fallback.command, fallback.args).then(resolve);
+      } else {
+        process.stderr.write(`itiger: unable to start ${command}: ${error.message}\n`);
+        resolve(1);
+      }
       return;
     }
 
     child.on('error', (error) => {
-      process.stderr.write(`itiger: unable to start ${command}: ${error.message}\n`);
-      resolve(1);
+      if (settled) return;
+      settled = true;
+      if (fallback && error.code === 'ENOENT') {
+        runCommand(fallback.command, fallback.args).then(resolve);
+      } else {
+        process.stderr.write(`itiger: unable to start ${command}: ${error.message}\n`);
+        resolve(1);
+      }
     });
     child.on('exit', (code, signal) => {
+      if (settled) return;
+      settled = true;
       resolve(code === null ? 128 + signalNumber(signal) : code);
     });
   });

package/src/skills-adapter.js

@@ -8,9 +8,9 @@ function mapRepository(repository) {
     return `https://${INTERNAL_GIT_HOST}/${githubUrl[1]}/${githubUrl[2]}.git`;
   }
 
-  const shorthand = repository.match(/^([^/]+)\/([^/]+)$/);
+  const shorthand = repository.match(/^([^/]+\/.+?)(?:\.git)?$/);
   if (shorthand) {
-    return `https://${INTERNAL_GIT_HOST}/${shorthand[1]}/${shorthand[2]}.git`;
+    return `https://${INTERNAL_GIT_HOST}/${shorthand[1]}.git`;
   }
 
   return repository;
@@ -32,7 +32,11 @@ function runSkills(args) {
     process.stderr.write('itiger: only "skills add" is supported through this adapter\n');
     return Promise.resolve(2);
   }
-  return runCommand('skills', transformArgs(args));
+  const transformedArgs = transformArgs(args);
+  return runCommand('skills', transformedArgs, {
+    command: 'npx',
+    args: ['--yes', 'skills', ...transformedArgs],
+  });
 }
 
 module.exports = { INTERNAL_GIT_HOST, mapRepository, transformArgs, runSkills };