pi-lens 3.6.1 → 3.6.3

package/CHANGELOG.md

@@ -2,6 +2,29 @@
 
 All notable changes to pi-lens will be documented in this file.
 
+## [3.6.2] - 2026-04-02
+
+### Added
+- **Condensed skill auto-loading** — Injects ~70-token tool selection guidance at session start (vs 1,355 for full skills):
+  - Quick reference for when to use lsp_navigation vs ast_grep_search vs grep
+  - References full skills for lazy loading (ast-grep, lsp-navigation)
+  - Prevents common tool selection errors without loading full skill content
+
+### Changed
+- **Streamlined session start injection** — Removed TODO/Knip/jscpd reports from initial context:
+  - Scans still run and cache for on-demand access via `/lens-booboo`
+  - Reduces session start noise (only active tools list, error reminder, skill guidance remain)
+  - Caching preserved for duplicate detection on file writes
+
+## [3.6.1] - 2026-04-02
+
+### Changed
+- **Updated package description** — More concise: "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo"
+
+### Repository
+- **AGENTS.md is now local-only** — Removed from git repo and added to `.gitignore` so it stays local to each developer's environment
+- **Cleaned up debug files** — Removed old test files (`_debug-*.ts`, `_trigger-test.ts`, `_test-*.ts`) from repo
+
 ## [3.6.0] - 2026-04-02
 
 ### Added

package/index.ts

@@ -944,6 +944,11 @@ export default function (pi: ExtensionAPI) {
 			log(`Active tools: ${tools.join(", ")}`);
 			dbg(`session_start tools: ${tools.join(", ")}`);
 
+			// --- Condensed skill guidance for efficient tool selection ---
+			// Full skills available at skills/ast-grep/SKILL.md and skills/lsp-navigation/SKILL.md
+			const condensedGuidance =
+				"## Code Tool Selection\n- Navigation (definitions, references): lsp_navigation\n- Pattern search (functions, imports): ast_grep_search\n- Text/TODOs: grep\n- Full guides: skills/ast-grep, skills/lsp-navigation";
+
 			const parts: string[] = [];
 
 			// --- Error ownership reminder ---
@@ -952,6 +957,9 @@ export default function (pi: ExtensionAPI) {
 				"📌 Remember: If you find ANY errors (test failures, compile errors, lint issues) in this codebase, fix them — even if you didn't cause them. Don't skip errors as 'not my fault'.",
 			);
 
+			// Add condensed skill guidance to system message
+			parts.push(condensedGuidance);
+
 			// Scan for project-specific rules (.claude/rules, .agents/rules, CLAUDE.md, etc.)
 			projectRulesScan = scanProjectRules(cwd);
 			if (projectRulesScan.hasCustomRules) {
@@ -969,13 +977,12 @@ export default function (pi: ExtensionAPI) {
 				dbg("session_start: no project rules found");
 			}
 
-			// TODO/FIXME scan — fast, no deps
+			// TODO/FIXME scan — fast, no deps (cached for on-demand reporting)
 			const todoResult = todoScanner.scanDirectory(cwd);
-			const todoReport = todoScanner.formatResult(todoResult);
 			dbg(`session_start TODO scan: ${todoResult.items.length} items`);
-			if (todoReport) parts.push(todoReport);
+			// Note: TODOs not shown at session start — use /lens-booboo to see them
 
-			// Dead code scan — use cache if fresh, auto-install if needed
+			// Dead code scan — use cache if fresh, auto-install if needed (cached for on-demand reporting)
 			if (await knipClient.ensureAvailable()) {
 				const cached = cacheManager.readCache<
 					ReturnType<KnipClient["analyze"]>
@@ -984,23 +991,20 @@ export default function (pi: ExtensionAPI) {
 					dbg(
 						`session_start Knip: cache hit (${Math.round((Date.now() - new Date(cached.meta.timestamp).getTime()) / 1000)}s ago)`,
 					);
-					const knipReport = knipClient.formatResult(cached.data);
-					if (knipReport) parts.push(knipReport);
 				} else {
 					const startMs = Date.now();
 					const knipResult = knipClient.analyze(cwd);
 					cacheManager.writeCache("knip", knipResult, cwd, {
 						scanDurationMs: Date.now() - startMs,
 					});
-					const knipReport = knipClient.formatResult(knipResult);
 					dbg(`session_start Knip scan done`);
-					if (knipReport) parts.push(knipReport);
 				}
 			} else {
 				dbg(`session_start Knip: not available`);
 			}
+			// Note: Knip results not shown at session start — use /lens-booboo to see dead code
 
-			// Duplicate code detection — use cache if fresh, auto-install if needed
+			// Duplicate code detection — use cache if fresh, auto-install if needed (cached for on-demand reporting)
 			if (await jscpdClient.ensureAvailable()) {
 				const cached = cacheManager.readCache<ReturnType<JscpdClient["scan"]>>(
 					"jscpd",
@@ -1009,8 +1013,6 @@ export default function (pi: ExtensionAPI) {
 				if (cached) {
 					dbg(`session_start jscpd: cache hit`);
 					_cachedJscpdClones = cached.data.clones;
-					const jscpdReport = jscpdClient.formatResult(cached.data);
-					if (jscpdReport) parts.push(jscpdReport);
 				} else {
 					const startMs = Date.now();
 					const jscpdResult = jscpdClient.scan(cwd);
@@ -1018,13 +1020,12 @@ export default function (pi: ExtensionAPI) {
 					cacheManager.writeCache("jscpd", jscpdResult, cwd, {
 						scanDurationMs: Date.now() - startMs,
 					});
-					const jscpdReport = jscpdClient.formatResult(jscpdResult);
 					dbg(`session_start jscpd scan done`);
-					if (jscpdReport) parts.push(jscpdReport);
 				}
 			} else {
 				dbg(`session_start jscpd: not available`);
 			}
+			// Note: jscpd results not shown at session start — use /lens-booboo to see duplicates
 
 			// Note: type-coverage runs on-demand via /lens-booboo only (not at session_start)
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-lens",
-	"version": "3.6.1",
+	"version": "3.6.3",
 	"type": "module",
 	"description": "Real-time code feedback for pi — LSP, linters, formatters, type-checking, structural analysis & booboo",
 	"repository": {
@@ -29,7 +29,7 @@
 		"jscpd",
 		"knip"
 	],
-	"author": "R3LiC",
+	"author": "Apostolos Mantzaris",
 	"license": "MIT",
 	"files": [
 		"index.ts",

package/skills/ast-grep/SKILL.md

@@ -143,6 +143,12 @@ pattern: "console.log($$$ARGS)"
 
 **Error: "Multiple AST nodes detected"** → Your pattern has multiple code fragments. Use metavariables like `$TEST` instead of literal text in quotes.
 
+**No matches found?** → Debug efficiently (don't `read` large files):
+1. Use `ast_grep_search` with same pattern to preview matches - costs same as `read` for small result sets
+2. Simplify pattern: remove constraints, test basic match first
+3. Check metavariables capture what you expect (`$PARAM` = whole parameter including name and type)
+4. Ensure quotes/parentheses balance in pattern
+
 **Fallback:** If pattern fails twice → `grep -rn "pattern" src/`
 
 **Debug:** https://ast-grep.github.io/playground.html