pi-lens 4.1.2 → 4.1.4

package/dist/clients/string-utils.js

@@ -0,0 +1,33 @@
+/**
+ * Ordinal (code-unit) string comparator for identity-feeding sorts — cache
+ * keys, dedupe keys, hashes, and signatures. Refs #2155, #2165.
+ *
+ * `localeCompare` orders strings by locale-dependent collation (the DEFAULT
+ * `Array.prototype.sort`, with no comparator, does not — it compares UTF-16
+ * code units, which is already ordinal). Locale-dependent order is fine for
+ * a list a human reads. It is a bug (SonarCloud S2871) when the sorted
+ * result feeds an identity: two processes — or one process whose locale
+ * changes — can order the same input differently under `localeCompare` and
+ * mint different keys for equivalent state, producing a silent cache or
+ * dedupe miss.
+ *
+ * Use this comparator whenever a sort's output becomes a key, a hash input,
+ * or a signature compared for equality later. Keep `localeCompare` for
+ * sorting that only affects what a person reads.
+ */
+export function compareOrdinal(a, b) {
+    return a < b ? -1 : a > b ? 1 : 0;
+}
+/**
+ * Escape a string for safe interpolation into a `RegExp` source, so every
+ * regex metacharacter it contains is matched literally instead of taking on
+ * its regex meaning. This is the ONE runtime copy (#2558) — string-utils.ts
+ * is a leaf (no imports of its own), so importing it never creates a cycle.
+ * Before #2558 this same body was hand-copied under this name (and as
+ * `escapeRegExpChar`/`escapeRegExpLiteral`) into ~8 production modules and 3
+ * test helpers; fold new call sites onto this export instead of re-copying
+ * it.
+ */
+export function escapeRegExp(value) {
+    return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+}

package/dist/clients/subagent-mode.js

@@ -4,8 +4,16 @@
  * Two known subagent-spawning ecosystems set different env vocabularies in
  * every spawned child's environment:
  *
- * - nicobailon/pi-subagents sets `PI_SUBAGENT_CHILD=1` unconditionally (plus
- *   `PI_SUBAGENT_RUN_ID` / `PI_SUBAGENT_CHILD_AGENT` for best-effort identity).
+ * - nicobailon/pi-subagents sets `PI_SUBAGENT_CHILD=1` unconditionally on
+ *   every process that hosts a child session — the only var light mode keys
+ *   on for this vocabulary. It ALSO used to set `PI_SUBAGENT_RUN_ID` /
+ *   `PI_SUBAGENT_CHILD_AGENT` alongside it for best-effort identity, but
+ *   removed both entirely in `pi-subagents@0.65.0`'s native-AgentSession
+ *   rewrite (grep-verified absent from the whole 0.66.0 source tree, #2581/
+ *   #2680) — child identity now travels through an in-process object, not
+ *   env vars. `getSubagentIdentity()` below already treats identity as
+ *   optional best-effort metadata, so this permanent absence for the
+ *   nicobailon vocabulary needed no behavior change, only the doc update.
  * - avtc-pi-subagent (the spawn engine under avtc-pi-feature-flow) is the same
  *   execution model — real child-process `pi --mode rpc` / `--mode json -p`
  *   spawns, full env inheritance — but never sets `PI_SUBAGENT_CHILD`. It sets
@@ -63,8 +71,13 @@ export function isSubagentSession() {
 /**
  * Best-effort identity of the current subagent, read from the env vars the
  * detected extension sets alongside its subagent marker. Returns `undefined`
- * when neither identity var is present (e.g. not a subagent session, or a
- * future extension that doesn't set them).
+ * when neither identity var is present — not a subagent session, an
+ * extension that never set them, OR nicobailon/pi-subagents@0.65.0+, which
+ * removed both identity vars upstream while still setting `PI_SUBAGENT_CHILD`
+ * (#2581/#2680): light mode still engages via `isSubagentSession()` alone,
+ * `marker` is still populated, only `runId`/`agentName` degrade to
+ * `undefined`. Identity is metadata for observability, never a light-mode
+ * gate — callers must not treat its absence as "not a subagent".
  */
 export function getSubagentIdentity() {
     const runId = process.env.PI_SUBAGENT_RUN_ID || undefined;