chisle 3.3.0 → 3.5.0

package/pi-extension/index.js

@@ -3,7 +3,7 @@
 const crypto = require('node:crypto');
 const fs = require('node:fs');
 const path = require('node:path');
-const { getDefaultMode } = require('../hooks/chisle-config');
+const { getDefaultMode, getSections, filterSections } = require('../hooks/chisle-config');
 const { duplicateMarker, limitsFor, transform } = require('../hooks/chisle-compress-output');
 const { requestedMode } = require('../hooks/chisle-mode');
 
@@ -12,20 +12,52 @@ const MODE_TYPE = 'chisle-mode';
 const SAFE_TOOLS = new Set(['bash', 'powershell', 'grep', 'find', 'ls']);
 const NEVER_COMPRESS = new Set(['read', 'edit', 'write']);
 const DEDUP_MIN = 2048;
-const FALLBACK_RULES = `CHISLE ACTIVE
+// Sectioned so the `sections` config applies here too. Flat prose/code lines
+// would leave the fallback ignoring a user's sections.prose:false the moment
+// SKILL.md became unreadable — chisle-activate.js's fallback is shaped the
+// same way for the same reason. The safety line stays outside both groups.
+const FALLBACK_BODY = `Chisle: maximum-efficiency dev mode. Zero-fluff prose. YAGNI-first code.
 
-Chisle: maximum-efficiency dev mode. Zero-fluff prose. YAGNI-first code.
+## Prose
 
 Drop articles, filler, pleasantries, and hedging. Fragments OK; technical terms exact.
+
+## Code
+
 Code ladder: YAGNI → reuse → stdlib → native → installed dependency → one line → minimum code.
+
+## Boundaries
+
 Never simplify away input validation, data-loss prevention, security, accessibility, or explicit requirements.`;
 
 function loadRules() {
   try {
     const skill = fs.readFileSync(path.join(__dirname, '..', 'skills', 'chisle', 'SKILL.md'), 'utf8');
-    return 'CHISLE ACTIVE\n\n' + skill.replace(/^---[\s\S]*?---\s*/, '');
+    // Shares the SKILL.md emission path with chisle-activate.js, so it honors
+    // the same `sections` config (prose/code rule suppression) the same way.
+    return 'CHISLE ACTIVE\n\n' +
+      filterSections(skill.replace(/^---[\s\S]*?---\s*/, ''), getSections());
+  } catch (_) {
+    return 'CHISLE ACTIVE\n\n' +
+      filterSections(FALLBACK_BODY, getSections(), ['Prose'], ['Code']);
+  }
+}
+
+// OMP (oh-my-pi) ships a subset of Pi's sessionManager: buildContextEntries and
+// getBranch are absent there, so calling one throws inside session_start and
+// takes the whole extension down —
+// "ctx.sessionManager.buildContextEntries is not a function" (issue #14).
+// Treat both as optional readers. An absent one means we cannot prove the
+// session state, and every caller here already behaves correctly on "unknown":
+// no mode entry leaves `active` at getDefaultMode(), and rulesInContext=false
+// re-injects the ruleset, which promptHasRules still de-duplicates.
+function sessionEntries(ctx, method) {
+  try {
+    const sm = ctx && ctx.sessionManager;
+    if (!sm || typeof sm[method] !== 'function') return [];
+    return sm[method]() || [];
   } catch (_) {
-    return FALLBACK_RULES;
+    return [];
   }
 }
 
@@ -77,12 +109,12 @@ function chisleExtension(pi) {
     lastByToolBlock = new Map();
     dedupBaseline = new Map();
 
-    const modeEntry = ctx.sessionManager.getBranch()
+    const modeEntry = sessionEntries(ctx, 'getBranch')
       .filter(entry => entry.type === 'custom' && entry.customType === MODE_TYPE).pop();
     if (modeEntry && modeEntry.data && ['on', 'off'].includes(modeEntry.data.mode)) {
       active = modeEntry.data.mode === 'on';
     }
-    rulesInContext = hasRulesEntry(ctx.sessionManager.buildContextEntries());
+    rulesInContext = hasRulesEntry(sessionEntries(ctx, 'buildContextEntries'));
     updateStatus(ctx);
   }
 
@@ -116,7 +148,7 @@ function chisleExtension(pi) {
   pi.on('session_compact', async (_event, ctx) => {
     lastByToolBlock.clear();
     dedupBaseline.clear();
-    rulesInContext = hasRulesEntry(ctx.sessionManager.buildContextEntries());
+    rulesInContext = hasRulesEntry(sessionEntries(ctx, 'buildContextEntries'));
   });
 
   pi.on('input', async (event, ctx) => {
@@ -185,4 +217,5 @@ function chisleExtension(pi) {
 module.exports = chisleExtension;
 module.exports.loadRules = loadRules;
 module.exports.hasRulesEntry = hasRulesEntry;
+module.exports.sessionEntries = sessionEntries;
 module.exports.piToolAllowed = piToolAllowed;