@marcomigozzi/trailhead 0.16.1 → 0.18.0

package/plugins/trailhead/templates/trailhead-uat-backfill.js

@@ -18,6 +18,7 @@
 const fs = require('fs');
 const path = require('path');
 const { spawnSync } = require('child_process');
+const { gitEnv } = require('./trailhead-git-env.js');
 
 const LABEL = 'trailhead:uat';
 const COLOR = 'BFD4F2';
@@ -27,7 +28,7 @@ const LIMIT = 1000;
 const UAT_LINE_RE = /^UAT of:/m;
 
 function defaultGh(args) {
-  const r = spawnSync('gh', args, { encoding: 'utf8', maxBuffer: 64 * 1024 * 1024 });
+  const r = spawnSync('gh', args, { encoding: 'utf8', env: gitEnv(), maxBuffer: 64 * 1024 * 1024 });
   return { status: r.status === null ? 1 : r.status, stdout: r.stdout || '', stderr: r.stderr || (r.error ? String(r.error.message) : '') };
 }
 

package/plugins/trailhead/templates/trailhead-uat-followup.js

@@ -25,13 +25,14 @@
 // Exit 0 on ok, 1 on an error, 2 on a usage error (stderr).
 
 const { spawnSync } = require('child_process');
+const { gitEnv } = require('./trailhead-git-env.js');
 
 const MAX_PAGES = 100; // 100 pages x 100 issues
 const QUERY = 'query($owner:String!,$name:String!,$after:String){repository(owner:$owner,name:$name){issues(first:100,after:$after,labels:["trailhead:uat"],states:[OPEN,CLOSED],orderBy:{field:CREATED_AT,direction:ASC}){nodes{number title state body} pageInfo{hasNextPage endCursor}}}}';
 const URL_RE = /https:\/\/github\.com\/([^/\s)]+)\/([^/\s)]+)\/issues\/(\d+)(?![\w])/gi;
 
 function defaultGh(args) {
-  const r = spawnSync('gh', args, { encoding: 'utf8', maxBuffer: 64 * 1024 * 1024 });
+  const r = spawnSync('gh', args, { encoding: 'utf8', env: gitEnv(), maxBuffer: 64 * 1024 * 1024 });
   return { status: r.status === null ? 1 : r.status, stdout: r.stdout || '', stderr: r.stderr || (r.error ? String(r.error.message) : '') };
 }
 

package/plugins/trailhead/templates/trailhead-uat-setup.js

@@ -46,6 +46,7 @@ const fs = require('fs');
 const os = require('os');
 const path = require('path');
 const { spawnSync } = require('child_process');
+const { gitEnv } = require('./trailhead-git-env.js');
 
 const CONFIG_REL = path.join('.trailhead', 'config.json');
 const OUTPUT_TAIL = 2000;
@@ -118,8 +119,12 @@ function pick(cfg, key, file) {
   return out;
 }
 
+// Su Codex lo script vive in <codexHome>/skills/trailhead/<dir>: la config globale sta lì, non in ~/.claude.
+const CODEX_INSTALL = /[\\/]skills[\\/]trailhead[\\/](?:hooks|templates)$/;
+const codexHome = () => (CODEX_INSTALL.test(__dirname) ? path.resolve(__dirname, '..', '..', '..') : null);
+
 function globalConfigPath(env) {
-  const base = env.CLAUDE_CONFIG_DIR || path.join(env.HOME || os.homedir(), '.claude');
+  const base = codexHome() || env.CLAUDE_CONFIG_DIR || path.join(env.HOME || os.homedir(), '.claude');
   return path.join(base, 'trailhead', 'config.json');
 }
 
@@ -257,7 +262,7 @@ function redactor(names, env) {
 function sh(command, repo, env, timeoutMs, wantOutput) {
   const timeout = Math.max(1, Math.ceil(timeoutMs));
   if (!wantOutput) {
-    const r = spawnSync('sh', ['-c', command], { cwd: repo, env, stdio: 'ignore', timeout, killSignal: 'SIGKILL' });
+    const r = spawnSync('sh', ['-c', command], { cwd: repo, env: gitEnv(env), stdio: 'ignore', timeout, killSignal: 'SIGKILL' });
     const timedOut = Boolean(r.error && r.error.code === 'ETIMEDOUT');
     return { code: r.status, timedOut, spawnError: r.error && !timedOut ? r.error.message : null, tail: '' };
   }
@@ -265,7 +270,7 @@ function sh(command, repo, env, timeoutMs, wantOutput) {
   const file = path.join(dir, 'out');
   const fd = fs.openSync(file, 'w');
   try {
-    const r = spawnSync('sh', ['-c', command], { cwd: repo, env, stdio: ['ignore', fd, fd], timeout, killSignal: 'SIGKILL' });
+    const r = spawnSync('sh', ['-c', command], { cwd: repo, env: gitEnv(env), stdio: ['ignore', fd, fd], timeout, killSignal: 'SIGKILL' });
     const timedOut = Boolean(r.error && r.error.code === 'ETIMEDOUT');
     const res = { code: r.status, timedOut, spawnError: r.error && !timedOut ? r.error.message : null, tail: '' };
     if (wantOutput) {
@@ -377,7 +382,7 @@ function main(argv, { cwd, stdout, env, fs: fsMod } = {}) {
   return 0;
 }
 
-module.exports = { parseArgs, readConfig, main };
+module.exports = { globalConfigPath, parseArgs, readConfig, main };
 
 if (require.main === module) {
   const code = main(process.argv.slice(2), { cwd: process.cwd(), stdout: process.stdout, env: process.env });

package/plugins/trailhead/templates/trailhead-unblock.js

@@ -21,11 +21,12 @@
 // Exit 0 on ok, 1 on an error, 2 on a usage error (stderr).
 
 const { spawnSync } = require('child_process');
+const { gitEnv } = require('./trailhead-git-env.js');
 
 const LABEL = 'trailhead:blocked';
 
 function defaultGh(args) {
-  const r = spawnSync('gh', args, { encoding: 'utf8', maxBuffer: 64 * 1024 * 1024 });
+  const r = spawnSync('gh', args, { encoding: 'utf8', env: gitEnv(), maxBuffer: 64 * 1024 * 1024 });
   return { status: r.status === null ? 1 : r.status, stdout: r.stdout || '', stderr: r.stderr || (r.error ? String(r.error.message) : '') };
 }
 

package/plugins/trailhead/tests/auto-run-ui.test.ts

@@ -1,5 +1,6 @@
 import type { On } from 'claude-code'
 import { expect, test } from 'claude-code/testing'
+import { unscrub } from './unscrub'
 
 import { resetAutoRun } from '../hooks/mod/auto-run'
 import { resetLocate } from '../hooks/mod/work-state-locate'
@@ -47,7 +48,7 @@ const rig = (on: On) => {
   on('session.cwd', () => ({ value: r.cwd }))
   on('session.id', () => ({ value: SESSION }))
   on('process.run', (_$, e) => {
-    const line = e.argv.join(' ')
+    const line = unscrub(e.argv).join(' ')
     r.runs.push(line)
     r.cwds.push(e.cwd ?? '')
     const out = (stdout: string, exitCode = 0) => ({ value: { exitCode, stdout, stderr: '', isStdoutTruncated: false, isStderrTruncated: false } })

package/plugins/trailhead/tests/git-env.test.ts

@@ -0,0 +1,20 @@
+import { expect, test } from 'claude-code/testing'
+
+import { GIT_LOCATION_VARS, gitArgv, scrubGitEnv } from '../hooks/mod/git-env'
+import { unscrub } from './unscrub'
+
+test('gitArgv wraps git in env -u for every repo-location variable', () => {
+  const argv = gitArgv('-C', '/repo', 'status')
+  expect(argv.slice(0, 3)).toEqual(['env', '-u', GIT_LOCATION_VARS[0]])
+  for (const name of GIT_LOCATION_VARS) expect(argv).toContain(name)
+  expect(argv.slice(-4)).toEqual(['git', '-C', '/repo', 'status'])
+  expect(unscrub(argv)).toEqual(['git', '-C', '/repo', 'status'])
+})
+
+test('GIT_DIR and GIT_WORK_TREE are scrubbed, GIT_CONFIG* is not', () => {
+  const argv = scrubGitEnv(['gh', 'api', 'graphql'])
+  expect(argv).toContain('GIT_DIR')
+  expect(argv).toContain('GIT_WORK_TREE')
+  expect(argv.some(a => a.startsWith('GIT_CONFIG'))).toBe(false)
+  expect(unscrub(argv)).toEqual(['gh', 'api', 'graphql'])
+})