@spences10/pi-tool-guidance 0.1.0 → 0.1.1

package/README.md

@@ -14,8 +14,8 @@
 ## Tool workflow guidance
 
 Distributes native Pi skills and concise system-prompt reminders for
-nopeek, Omnisearch, and SQLite. It does not depend on the maintainer's
-local files.
+nopeek, Omnisearch, SQLite, and language-aware validation. It does not
+depend on the maintainer's local files.
 
 ## What ships
 
@@ -37,6 +37,57 @@ conditional on reachable tools or Pi's configured indirect-server
 listing. Both codemode and tool-search exposure work without requiring
 direct tool declarations.
 
+## Language-aware validation
+
+When editing and shell tools are reachable, the prompt reminds the
+agent to use the project's configured linter, type checker, Svelte
+tooling, and tests for code correctness. Coding-preference regexes
+remain workflow/text policies; Svelte guardrails block `$effect` by
+default, with explicit user-controlled exceptions. No parser,
+classifier gate, new validation tool, or automatic command execution
+is added.
+
+Use native Codemode to coordinate existing tools:
+
+1. Read project instructions and package scripts to select the
+   affected checks; do not assume every project uses the same
+   commands.
+2. Apply edits through `tools.edit()` / `tools.write()` and await them
+   before checking. Native nested calls retain the same guardrail
+   hooks as direct calls; do not scan the outer JavaScript as source
+   code.
+3. Run independent checks together only when they do not share mutable
+   outputs. Keep dependent edits, builds, and checks sequential.
+4. Inspect exit codes and tool errors, then summarize diagnostics. Do
+   not infer success from a fulfilled promise: `bash` resolves even
+   when its command exits nonzero.
+5. Correct the findings and rerun affected checks before reporting
+   completion. Code validation runs after the edit; it is not an
+   atomic pre-write security boundary.
+
+For example, after choosing this repository's `check` script:
+
+```js
+const result = await tools.bash({
+	command: 'pnpm run check',
+	timeout: 120,
+});
+text({
+	exit_code: result.exit_code,
+	truncated: result.truncated,
+	output: result.output.slice(-12000),
+	full_output_path: result.full_output_path,
+});
+if (result.exit_code !== 0) throw new Error('Project checks failed');
+```
+
+Inspect omitted output before diagnosing a failure. For multiple
+independent tool calls, `Promise.allSettled()` preserves successful
+results when another call rejects; inspect each result as well as each
+command's exit code. Codemode is orchestration, not a syntax checker
+or a way around guardrails. No classifier or paid model call is
+needed.
+
 ## User configuration remains yours
 
 The package does not install the nopeek CLI, start MCP servers, set

package/dist/index.js

@@ -4,6 +4,8 @@ const OMNISEARCH_GUIDANCE = `## Web research
 For research and current external facts, prefer the configured Omnisearch MCP server. Read the bundled my-pi-omnisearch skill. Discover its actual tools with tool search or codemode, and read source content before presenting claims; snippets alone are not verification.`;
 const SQLITE_GUIDANCE = `## SQLite databases
 Prefer the configured SQLite MCP tools over ad-hoc shell SQL. Read the bundled my-pi-sqlite skill; inspect schema before querying, keep result limits small, and prefer a backup plus an explicit transaction before authorized writes. Do not bypass safety gates to make destructive operations easier.`;
+const VALIDATION_GUIDANCE = `## Language-aware validation
+Use the project's configured linter, type checker, Svelte tooling, and tests for naming and language correctness. Regex guardrails enforce workflow or explicit local policy, not syntax. After edits, run the affected checks. When codemode is available, use it to coordinate independent checks and summarize diagnostics; inspect command exit codes and tool errors before reporting success. Keep edits dependent on earlier results sequential.`;
 export function tool_guidance(options, tools) {
     const selected = options.selectedTools ?? [
         'read',
@@ -30,8 +32,11 @@ export function tool_guidance(options, tools) {
     const available = `${names}\n${servers}`.toLowerCase();
     const shell = selected.some((name) => ['bash', 'powershell'].includes(name)) ||
         reachable.some((tool) => ['bash', 'powershell'].includes(tool.name));
+    const editing = selected.some((name) => ['edit', 'write'].includes(name)) ||
+        reachable.some((tool) => ['edit', 'write'].includes(tool.name));
     return [
         shell ? NOPEEK_GUIDANCE : '',
+        shell && editing ? VALIDATION_GUIDANCE : '',
         available.includes('omnisearch') ? OMNISEARCH_GUIDANCE : '',
         available.includes('sqlite') ? SQLITE_GUIDANCE : '',
     ]

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAUA,MAAM,eAAe,GAAG;0VACkU,CAAC;AAE3V,MAAM,mBAAmB,GAAG;8QACkP,CAAC;AAE/Q,MAAM,eAAe,GAAG;0SACkR,CAAC;AAE3S,MAAM,UAAU,aAAa,CAC5B,OAAiC,EACjC,KAAsB;IAEtB,MAAM,QAAQ,GAAG,OAAO,CAAC,aAAa,IAAI;QACzC,MAAM;QACN,MAAM;QACN,MAAM;QACN,OAAO;KACP,CAAC;IACF,MAAM,QAAQ,GACb,QAAQ,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC;IACnE,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAC7B,CAAC,IAAI,EAAE,EAAE,CACR,IAAI,CAAC,QAAQ,KAAK,QAAQ;QAC1B,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC;YAC5B,CAAC,QAAQ;gBACR,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,CACrD,CAAC;IACF,MAAM,KAAK,GAAG,SAAS;SACrB,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC;QAClB,IAAI,CAAC,IAAI;QACT,IAAI,CAAC,SAAS,EAAE,IAAI,IAAI,EAAE;QAC1B,GAAG,CAAC,IAAI,CAAC,SAAS,EAAE,OAAO,IAAI,EAAE,CAAC;KAClC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAC;IACb,qFAAqF;IACrF,MAAM,OAAO,GAAG,QAAQ;QACvB,CAAC,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,WAAW,IAAI,EAAE,CAAC;QACvC,CAAC,CAAC,EAAE,CAAC;IACN,MAAM,SAAS,GAAG,GAAG,KAAK,KAAK,OAAO,EAAE,CAAC,WAAW,EAAE,CAAC;IACvD,MAAM,KAAK,GACV,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC9D,SAAS,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CACvB,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAC1C,CAAC;IACH,OAAO;QACN,KAAK,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,EAAE;QAC5B,SAAS,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,EAAE;QAC3D,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,EAAE;KACnD;SACC,MAAM,CAAC,OAAO,CAAC;SACf,IAAI,CAAC,MAAM,CAAC,CAAC;AAChB,CAAC;AAED,MAAM,CAAC,OAAO,UAAU,uBAAuB,CAC9C,EAAgB;IAEhB,EAAE,CAAC,EAAE,CAAC,oBAAoB,EAAE,CAAC,KAAK,EAAE,EAAE;QACrC,MAAM,QAAQ,GAAG,aAAa,CAC7B,KAAK,CAAC,mBAAmB,EACzB,EAAE,CAAC,WAAW,EAAE,CAChB,CAAC;QACF,IAAI,QAAQ;YACX,KAAK,CAAC,mBAAmB,CAAC,QAAQ,CAAC,mBAAmB;gBACrD,QAAQ,CAAC;;YAEV,OAAO,KAAK,CAAC,mBAAmB,CAAC,QAAQ,CAAC,mBAAmB,CAAC;IAChE,CAAC,CAAC,CAAC;AACJ,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAUA,MAAM,eAAe,GAAG;0VACkU,CAAC;AAE3V,MAAM,mBAAmB,GAAG;8QACkP,CAAC;AAE/Q,MAAM,eAAe,GAAG;0SACkR,CAAC;AAE3S,MAAM,mBAAmB,GAAG;wbAC4Z,CAAC;AAEzb,MAAM,UAAU,aAAa,CAC5B,OAAiC,EACjC,KAAsB;IAEtB,MAAM,QAAQ,GAAG,OAAO,CAAC,aAAa,IAAI;QACzC,MAAM;QACN,MAAM;QACN,MAAM;QACN,OAAO;KACP,CAAC;IACF,MAAM,QAAQ,GACb,QAAQ,CAAC,QAAQ,CAAC,UAAU,CAAC,IAAI,QAAQ,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC;IACnE,MAAM,SAAS,GAAG,KAAK,CAAC,MAAM,CAC7B,CAAC,IAAI,EAAE,EAAE,CACR,IAAI,CAAC,QAAQ,KAAK,QAAQ;QAC1B,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC;YAC5B,CAAC,QAAQ;gBACR,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,CACrD,CAAC;IACF,MAAM,KAAK,GAAG,SAAS;SACrB,OAAO,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC;QAClB,IAAI,CAAC,IAAI;QACT,IAAI,CAAC,SAAS,EAAE,IAAI,IAAI,EAAE;QAC1B,GAAG,CAAC,IAAI,CAAC,SAAS,EAAE,OAAO,IAAI,EAAE,CAAC;KAClC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAC;IACb,qFAAqF;IACrF,MAAM,OAAO,GAAG,QAAQ;QACvB,CAAC,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,WAAW,IAAI,EAAE,CAAC;QACvC,CAAC,CAAC,EAAE,CAAC;IACN,MAAM,SAAS,GAAG,GAAG,KAAK,KAAK,OAAO,EAAE,CAAC,WAAW,EAAE,CAAC;IACvD,MAAM,KAAK,GACV,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC9D,SAAS,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CACvB,CAAC,MAAM,EAAE,YAAY,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAC1C,CAAC;IACH,MAAM,OAAO,GACZ,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QACzD,SAAS,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;IACjE,OAAO;QACN,KAAK,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,EAAE;QAC5B,KAAK,IAAI,OAAO,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,EAAE;QAC3C,SAAS,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,EAAE;QAC3D,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,EAAE;KACnD;SACC,MAAM,CAAC,OAAO,CAAC;SACf,IAAI,CAAC,MAAM,CAAC,CAAC;AAChB,CAAC;AAED,MAAM,CAAC,OAAO,UAAU,uBAAuB,CAC9C,EAAgB;IAEhB,EAAE,CAAC,EAAE,CAAC,oBAAoB,EAAE,CAAC,KAAK,EAAE,EAAE;QACrC,MAAM,QAAQ,GAAG,aAAa,CAC7B,KAAK,CAAC,mBAAmB,EACzB,EAAE,CAAC,WAAW,EAAE,CAChB,CAAC;QACF,IAAI,QAAQ;YACX,KAAK,CAAC,mBAAmB,CAAC,QAAQ,CAAC,mBAAmB;gBACrD,QAAQ,CAAC;;YAEV,OAAO,KAAK,CAAC,mBAAmB,CAAC,QAAQ,CAAC,mBAAmB,CAAC;IAChE,CAAC,CAAC,CAAC;AACJ,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spences10/pi-tool-guidance",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Packaged nopeek, Omnisearch and SQLite workflow guidance for Pi",
   "keywords": [
     "pi",