pi-tandem 0.1.0 → 0.3.0

package/README.md

@@ -18,6 +18,17 @@ Modern coding agents lean toward autonomy: multi-file changes in one go, subagen
 - **`subagent` skill** to run a task in a fresh agent session on request, with full visibility and control, instead of harness-managed subagents
 - **Tool-specific instructions** (`gh`, `aws`, `jira`, ...), added to the prompt only if the tool is actually installed
 
+## Recommended tools
+
+The plugin notices which CLI tools are actually installed on your machine, and for each one it finds, adds a short note telling the model that the tool is available and recommended to use. Installing these gets you significantly more out of this package:
+
+- `gh` - GitHub CLI: repos, issues, PRs, releases, workflows
+- `pandoc` - read docx/odt/rtf/html (and fetched web pages) as text instead of raw markup
+- `pdftotext` (poppler) - extract text from PDFs
+- a Chromium-based browser (Brave, Chromium, Chrome, Edge) - headless-render fallback for JS-heavy or bot-blocked pages
+- `osascript` (macOS only) - drive the user's real browser sessions via AppleScript
+- `jira`, `aws`, `saml2aws` - if your workflow includes them
+
 ## Attribution
 
 The "lazy senior" part of the coding section in the prompt is adapted from [ponytail](https://github.com/DietrichGebert/ponytail) by DietrichGebert (MIT).

package/extensions/tandem/index.ts

@@ -0,0 +1,6 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import injectProjectContext from "./pi-tandem";
+
+export default function (pi: ExtensionAPI) {
+  injectProjectContext(pi);
+}

package/extensions/{pi-tandem.ts → tandem/pi-tandem.ts}

@@ -1,11 +1,11 @@
 import { readFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
-import { filterCliTools } from "../runtime/cli-tools.mjs";
+import { filterCliTools } from "../../runtime/cli-tools.mjs";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 const prompt = filterCliTools(
-  readFileSync(join(dirname(fileURLToPath(import.meta.url)), "../prompt.md"), "utf8"),
+  readFileSync(join(dirname(fileURLToPath(import.meta.url)), "../../prompt.md"), "utf8"),
 );
 
 export default function (pi: ExtensionAPI) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-tandem",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/skhoroshavin/tandem-skills.git"
@@ -9,6 +9,7 @@
   "keywords": [
     "pi-package"
   ],
+  "type": "module",
   "license": "MIT",
   "files": [
     "extensions/",
@@ -17,5 +18,10 @@
     "prompt.md",
     "LICENSE",
     "README.md"
-  ]
+  ],
+  "devDependencies": {
+    "@earendil-works/pi-coding-agent": "^0.84.4",
+    "@types/node": "^26.4.1",
+    "typescript": "^7.0.2"
+  }
 }

package/prompt.md

@@ -43,6 +43,17 @@ A number of CLI tools are installed on this laptop and fully authenticated, you'
 <!--cli:saml2aws-->
 - AWS credentials are short-lived: on an expired or invalid token error, ask the user to run saml2aws login --idp-account <profile> --skip-prompt themselves, giving them the full command to copy-paste
 <!--/cli-->
+<!--cli:pdftotext-->
+- Use `pdftotext` to extract text from PDFs: `pdftotext input.pdf output.txt`, or `pdftotext input.pdf -` to print to stdout
+<!--/cli-->
+<!--cli:pandoc-->
+- Use `pandoc` to read documents other than pdf as markdown or plain text: `pandoc input.docx -o output.md` (also works for odt, rtf, html, ...)
+- Unless your task requires seeing actual tags, always convert HTML fetched from the web to plain text or markdown, instead of reading it raw, for example when using `curl`: `curl -s <url> | pandoc -f html -t plain`.
+<!--/cli-->
+<!--cli:browser-->
+- Use a real headless browser when you need to read a JS-rendered or bot-blocked page: `<browser-binary> --headless --disable-gpu --user-data-dir=$(mktemp -d) --dump-dom --virtual-time-budget=5000 <url>`
+- When in doubt, always try `curl` first before resorting to a headless browser
+<!--/cli-->
 <!--cli:osascript-->
 - Use `osascript` with `execute <tab> javascript "<js>"` to read pages and interact using the user's
   real logged-in sessions (analyzing dashboards, checking Google Calendar and Mail, etc)

package/runtime/cli-tools.mjs

@@ -18,18 +18,41 @@ export function isOnPath(binary) {
   return false;
 }
 
+// Chromium-family binary for the headless-render fallback; PATH first, then app bundles
+function findBrowserBinary() {
+  for (const name of ["brave", "brave-browser", "chromium", "chromium-browser", "google-chrome", "google-chrome-stable"]) {
+    if (isOnPath(name)) return name;
+  }
+  if (process.platform === "darwin") {
+    for (const app of ["Brave Browser", "Chromium", "Google Chrome", "Microsoft Edge"]) {
+      const bin = `/Applications/${app}.app/Contents/MacOS/${app}`;
+      try {
+        if (statSync(bin).isFile()) return bin;
+      } catch {}
+    }
+  }
+}
+
 // drop <!--cli:tool--> blocks for absent tools, and the whole section when none remain
+function toolAvailable(tool, browser) {
+  if (tool === "browser") return browser !== undefined;
+  if (tool === "osascript") return process.platform === "darwin" && isOnPath(tool);
+  return isOnPath(tool);
+}
+
 export function filterCliTools(raw) {
-  let any = false;
-  const filtered = raw.replace(
-    /<!--cli:([a-z0-9]+)-->\n([\s\S]*?)<!--\/cli-->\n?/g,
-    (_marker, tool, body) => {
-      const available =
-        tool === "osascript" ? process.platform === "darwin" && isOnPath(tool) : isOnPath(tool);
-      if (!available) return "";
-      any = true;
-      return body;
-    },
-  ).replace(/\n{3,}/g, "\n\n");
-  return any ? filtered : filtered.replace(/\n## CLI tools[\s\S]*?(?=\n## )/, "");
+  let hasTools = false;
+  const browser = findBrowserBinary();
+  const filtered = raw
+    .replace(
+      /<!--cli:([a-z0-9]+)-->\r?\n?([\s\S]*?)<!--\/cli-->\r?\n?/g,
+      (_marker, tool, body) => {
+        if (!toolAvailable(tool, browser)) return "";
+        hasTools = true;
+        return body;
+      },
+    )
+    .replace(/<browser-binary>/g, () => (browser ? JSON.stringify(browser) : "<browser-binary>"))
+    .replace(/\n{3,}/g, "\n\n");
+  return hasTools ? filtered : filtered.replace(/\n## CLI tools[\s\S]*?(?=\n## )/, "");
 }

package/skills/code-review/SKILL.md

@@ -0,0 +1,21 @@
+---
+name: code-review
+description: Use when asked to do a code review.
+---
+
+Read the full diff and everything it touches before judging; the worst review
+mistakes come from reviewing the diff only.
+
+Report rules:
+
+- Actionable findings only, ordered by severity: ones requiring changes, or
+  at least explicit decisions; "checked and fine" does not qualify.
+- Each finding: what is wrong and why it matters in 1-3 lines, then the
+  smallest fix - as a diff if it fits within 50 LoC, otherwise described
+  generally, asking for a decision.
+- No praise, no diff summary, no restating the request, no advice beyond
+  findings.
+- If nothing actionable: say exactly that, plus one line on what was checked.
+- Before writing, verify: grep for references to removed or renamed
+  identifiers; where the repo has rendered build output, check it is in sync;
+  diff the commit message claims against the actual change.