itiger 0.1.0 → 0.1.3

package/README.md

@@ -10,16 +10,19 @@ npx itiger install -g @tiger/example
 
 `skills add owner/repo` is translated to
 `https://git.tigerbrokers.net/owner/repo.git`; Git authentication remains under
-the user's normal SSH keys or Git credential helper. Other `skills` arguments
-and output are passed through unchanged.
+the user's normal SSH keys or Git credential helper. All other `skills`
+commands, arguments, output, and exit codes are passed through unchanged.
 
 When installing an `@tiger/*` package, `itiger` adds
 `--@tiger:registry=http://r.npm.tigerfintech.com` unless `--registry` or an
 explicit `--@tiger:registry` option is already present. Non-Tiger packages are
-delegated without registry changes.
+delegated without registry changes. Other npm commands are delegated with all
+arguments, output, and exit codes unchanged.
 
-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.3",
   "description": "A thin adapter for internal skills and @tiger npm packages.",
   "type": "commonjs",
   "bin": {

package/src/cli.js

@@ -8,12 +8,12 @@ async function main() {
   if (args[0] === 'skills') {
     return runSkills(args.slice(1));
   }
-  if (args[0] === 'install' || args[0] === 'i') {
+  if (args.length > 0) {
     return runNpm(args);
   }
 
-  process.stderr.write('Usage: itiger skills add <owner/repo> [skills options...]\n');
-  process.stderr.write('       itiger install [npm options...] <package...>\n');
+  process.stderr.write('Usage: itiger skills <skills command> [skills options...]\n');
+  process.stderr.write('       itiger <npm command> [npm options...]\n');
   return 2;
 }
 

package/src/npm-adapter.js

@@ -16,6 +16,7 @@ function hasTigerPackage(args) {
 }
 
 function transformArgs(args) {
+  if (args[0] !== 'install' && args[0] !== 'i') return [...args];
   if (!hasTigerPackage(args) || hasExplicitRegistry(args)) return [...args];
   return [...args, `--@tiger:registry=${TIGER_REGISTRY}`];
 }

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;
@@ -28,11 +28,11 @@ function transformArgs(args) {
 }
 
 function runSkills(args) {
-  if (args.length === 0 || args[0] !== 'add') {
-    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 };