Skip to content

Fix tools directory for pai alias#1267

Merged
danielmiessler merged 1 commit into
danielmiessler:mainfrom
maxolasersquad:tools-dir-fix
May 20, 2026
Merged

Fix tools directory for pai alias#1267
danielmiessler merged 1 commit into
danielmiessler:mainfrom
maxolasersquad:tools-dir-fix

Conversation

@maxolasersquad

Copy link
Copy Markdown
Contributor

The current code adds the alias alias pai='bun <home-dir>/.claude/PAI/Tools/pai.ts'. However the Tools directory is actually TOOLS. This creates a broken alias on Linux.

@Seikilos

Seikilos commented May 17, 2026

Copy link
Copy Markdown

Did you verify everything else works afterwards?

❯ grep -R "Tools" 2>/dev/null | wc -l
533
❯ grep -R "TOOLS" 2>/dev/null | wc -l
309

There are a lot of casing issues right now.
Wouldn't it make more sense to sweep all files and replace TOOLS with Tools?

I really just think of symlinking Tools to TOOLS 😅

@danielmiessler danielmiessler merged commit da07a11 into danielmiessler:main May 20, 2026
larsboes added a commit to larsboes/PAI that referenced this pull request Jun 10, 2026
…, PULSE casing

Selective integration of upstream danielmiessler/PAI (9fb9c862fde1bb):
- security: port execSync→execFileSync in tab-setter.ts (danielmiessler#1046), incl. the
  fork's extra cmux call sites + the kitten|jq pipe → execFileSync+JSON.parse,
  and replace `command -v` with `which`. Adds KITTY_LISTEN_ON socket validation.
- security: remove Midjourney/Discord integration from Art/Media skills (6 files)
  + strip stale "Midjourney" trigger keyword from Media SKILL.md descriptions.
- fix(PULSE): "Pulse"→"PULSE" directory casing across PULSE module — real bug on
  Linux/WSL2 case-sensitive FS (live dir is PULSE; refs were "Pulse") (danielmiessler#1259/danielmiessler#1175).
- bump .pai-fork/last-synced.ref → 2fde1bb.

Skipped (verified N/A to fork): plansDirectory danielmiessler#672 (no such key), case-colliding
danielmiessler#621 (no pai-observability-server), wiki Algorithm danielmiessler#1273 (fork casing consistent),
PAI-Install danielmiessler#1267 (fork doesn't deploy PAI-Install).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants