troxy-cli 1.30.21 → 1.30.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "troxy-cli",
-  "version": "1.30.21",
+  "version": "1.30.22",
   "description": "Make your Claude Code usage go further: lighter requests, less usage, less money",
   "homepage": "https://troxy.io",
   "bugs": {

package/src/init.js

@@ -325,14 +325,14 @@ async function printHostedAgentSetup(key) {
 // thing as a bare ~/AGENTS.md, which (per our own testing) a project working
 // directory does not walk up to. This is the one path that actually applies
 // everywhere, so it's the file the "all projects" option below writes to.
-function globalClaudeMdPath() {
+export function globalClaudeMdPath() {
   return path.join(os.homedir(), '.claude', 'CLAUDE.md');
 }
 
 // Same merge algorithm as writeAgentsFile (marker-delimited, never clobbers
 // content outside its own span), generalized to any file path so both the
 // per-project AGENTS.md and the global CLAUDE.md can share it.
-function writeMarkedBlock(file, text, startMarker, endMarker) {
+export function writeMarkedBlock(file, text, startMarker, endMarker) {
   const block = `${startMarker}\n${text}\n${endMarker}`;
   let existing = '';
   try { existing = fs.readFileSync(file, 'utf8'); } catch {}
@@ -352,7 +352,7 @@ function writeMarkedBlock(file, text, startMarker, endMarker) {
   return file;
 }
 
-function removeMarkedBlock(file, startMarker, endMarker) {
+export function removeMarkedBlock(file, startMarker, endMarker) {
   let existing;
   try { existing = fs.readFileSync(file, 'utf8'); } catch { return false; }
   if (!existing.includes(startMarker) || !existing.includes(endMarker)) return false;

package/src/slim.js

@@ -1,7 +1,7 @@
 import fs   from 'fs';
 import os   from 'os';
 import path from 'path';
-import { claudeCodeSettingsPath } from './init.js';
+import { claudeCodeSettingsPath, globalClaudeMdPath, writeMarkedBlock, removeMarkedBlock } from './init.js';
 import { loadConfig, saveConfig } from './config.js';
 import { api } from './api.js';
 
@@ -21,9 +21,9 @@ import { api } from './api.js';
 // of the tool in a transcript; tool definitions are in transcripts too, so
 // matching the bare name would find every session.
 export const TOOLS = [
-  { id: 'chrome',   label: 'Claude in Chrome (lets Claude control your browser)', tokens: 7200,
+  { id: 'chrome',   label: 'Claude in Chrome (lets Claude control your browser)', tokens: 7200, asks: 'to use or control a web browser',
     used: /"type":"tool_use","id":"[^"]+","name":"mcp__claude-in-chrome__/ },
-  { id: 'artifact', label: 'Artifacts (lets Claude publish pages to claude.ai)', tokens: 7500,
+  { id: 'artifact', label: 'Artifacts (lets Claude publish pages to claude.ai)', tokens: 7500, asks: 'for an artifact or a page published to claude.ai',
     used: /"type":"tool_use","id":"[^"]+","name":"Artifact"/ },
 ];
 
@@ -57,6 +57,23 @@ export function setToolOff(id, off, file = claudeCodeSettingsPath()) {
     throw new Error(`Unknown tool "${id}". Try: ${TOOLS.map(t => t.id).join(', ')}`);
   }
   write(file, s);
+  if (file === claudeCodeSettingsPath()) syncOffNote(file);
+}
+
+const NOTE_START = '<!-- troxy:slim:start -->';
+const NOTE_END   = '<!-- troxy:slim:end -->';
+
+// A turned-off tool is simply gone from Claude's tool list, so nothing errors:
+// Claude would only say it can't do that. This note in the global CLAUDE.md
+// tells Claude why and what to tell the user. About 50 tokens per tool, vs
+// the ~7k the tool itself costs.
+export function syncOffNote(file = claudeCodeSettingsPath(), noteFile = globalClaudeMdPath()) {
+  const off = TOOLS.filter(t => toolIsOff(t.id, file));
+  if (!off.length) { removeMarkedBlock(noteFile, NOTE_START, NOTE_END); return; }
+  writeMarkedBlock(noteFile, '## Turned off by Troxy\n\n' + off.map(t =>
+    `- ${t.label.split(' (')[0]} is turned off by Troxy to make every call smaller. If the user asks you ${t.asks}, ` +
+    `tell them it is off and that they can turn it back on by running \`troxy slim ${t.id} --undo\` in a terminal and then starting a new Claude Code session.`
+  ).join('\n'), NOTE_START, NOTE_END);
 }
 
 export function printToolsStatus(file = claudeCodeSettingsPath()) {
@@ -102,7 +119,9 @@ export async function reportSlim(apiKey = loadConfig()?.apiKey, file = claudeCod
 // back on with --undo stays on.
 export async function autoSlim({ file = claudeCodeSettingsPath(), projectsDir } = {}) {
   const cfg = loadConfig();
-  if (!cfg || cfg.slimChecked) return;
+  if (!cfg) return;
+  if (file === claudeCodeSettingsPath()) syncOffNote(file);  // also for tools turned off before the note existed
+  if (cfg.slimChecked) return;
   const used = usedTools(projectsDir);
   for (const t of TOOLS) {
     if (used.has(t.id) || toolIsOff(t.id, file)) continue;

package/src/tests/slim.test.js

@@ -4,7 +4,7 @@ import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 
-import { TOOLS, toolIsOff, setToolOff, usedTools } from '../slim.js';
+import { TOOLS, toolIsOff, setToolOff, usedTools, syncOffNote } from '../slim.js';
 
 // Turning a tool off and back on must leave the user's own settings.json
 // exactly as it was, including their own env vars and deny rules.
@@ -53,4 +53,18 @@ describe('troxy slim', () => {
     assert.deepEqual([...usedTools(path.join(dir, 'projects'))], ['chrome']);
     assert.deepEqual([...usedTools(path.join(dir, 'missing'))], []);
   });
+
+  it('keeps a note for Claude listing only the tools that are off, and keeps the user text', () => {
+    const note = path.join(dir, 'CLAUDE.md');
+    fs.writeFileSync(note, '# mine\n');
+    setToolOff('chrome', true, file);
+    syncOffNote(file, note);
+    let text = fs.readFileSync(note, 'utf8');
+    assert.match(text, /^# mine/);
+    assert.match(text, /troxy slim chrome --undo/);
+    assert.doesNotMatch(text, /troxy slim artifact/);
+    setToolOff('chrome', false, file);
+    syncOffNote(file, note);
+    assert.equal(fs.readFileSync(note, 'utf8').trim(), '# mine');
+  });
 });