@lincoln504/pi-research 1.0.5 → 1.0.6

package/dist/cli.mjs

@@ -20428,11 +20428,11 @@ init_constants();
 // src/orchestration/headless-observer.ts
 init_logger();
 function makeSafeObserver(observer) {
-  return new Proxy(observer, {
-    get(target, prop) {
+  return new Proxy({}, {
+    get(_target, prop) {
       let value;
       try {
-        value = Reflect.get(target, prop, target);
+        value = Reflect.get(observer, prop, observer);
       } catch (err) {
         logger.warn(`[observer] Ignored error reading observer.${String(prop)}:`, err);
         return void 0;
@@ -20440,7 +20440,7 @@ function makeSafeObserver(observer) {
       if (typeof value !== "function") return value;
       return (...args) => {
         try {
-          const out = value.apply(target, args);
+          const out = value.apply(observer, args);
           if (out && typeof out.then === "function") {
             return out.catch((err) => {
               logger.warn(`[observer] Ignored rejection from observer.${String(prop)}:`, err);
@@ -20452,6 +20452,17 @@ function makeSafeObserver(observer) {
           return void 0;
         }
       };
+    },
+    has(_target, prop) {
+      try {
+        return Reflect.has(observer, prop);
+      } catch {
+        return false;
+      }
+    },
+    // Keep `instanceof` transparent (legal on an extensible target).
+    getPrototypeOf() {
+      return Reflect.getPrototypeOf(observer);
     }
   });
 }

package/docs/ARCHITECTURE.md

@@ -25,8 +25,8 @@ pi CLI
    in-session, the calling agent picks the depth (1–3) from the user's wording and the
    task's complexity, guided by the tool's usage prompt (`src/prompts/research-tool-usage.md`).
    CLI and SDK callers pass the depth explicitly.
-2. Depth 0 takes the quick path; depth 1–3 takes the deep path (below). The TUI and agent
-   skill are restricted to levels 1–3.
+2. Depth 0 takes the quick path; depth 1–3 takes the deep path (below). The pi extension's
+   tool and the TUI are restricted to levels 1–3; the CLI, SDK, and agent skill can pass 0.
 3. On the deep path the coordinator plans the research tracks and runs one initial
    search burst, then hands each researcher a set of result URLs to start from.
 4. Researchers scrape and read those pages through the scrape tooling and return cited
@@ -154,7 +154,8 @@ WriterQueue (async, non-blocking)
 Key files: `src/knowledge/store.ts` (LanceDB operations), `embedder.ts` (model load +
 batched inference), `writer-queue.ts` (async writes + content-hash dedup), `chunker.ts`
 (chunking), `webgpu-viability.ts` (out-of-process GPU probe + cached verdict),
-`migration.ts` (model-change migration: drop / backup / re-embed).
+`migration.ts` (migration strategy types — the drop / backup / re-embed logic
+itself lives in `store.ts`).
 
 The store needs native ONNX-runtime and LanceDB bindings. On platforms with no prebuilt
 binary — notably Intel macOS (`darwin-x64`) — it is absent: the health check reports it

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lincoln504/pi-research",
-  "version": "1.0.5",
+  "version": "1.0.6",
   "description": "A set of web research tools (search, scrape, and others) and research agent system that uses them safely and efficiently.",
   "type": "module",
   "private": false,

package/skills/pi-research/SKILL.md

@@ -2,7 +2,7 @@
 name: pi-research
 description: "Multi-agent web research for any coding agent. Use for anything requiring current information from the internet — news, trends, product/service/company facts, statistics, people, places, events, APIs, library behavior, or 'what is X / how does X work' questions. Also use to search a local knowledge store of previously researched findings before doing live research. Powered by the pi-research engine (search, stealth scraping, security databases, Stack Exchange). Not for reading local files, running project commands, or analyzing this repository's own source."
 license: MIT
-metadata: { "author": "Lincoln Deen", "version": "1.0.5", "package": "@lincoln504/pi-research", "engine": "pi-research-sdk", "openclaw": { "requires": { "bins": ["node"] } } }
+metadata: { "author": "Lincoln Deen", "version": "1.0.6", "package": "@lincoln504/pi-research", "engine": "pi-research-sdk", "openclaw": { "requires": { "bins": ["node"] } } }
 allowed-tools: Bash(node:*)
 ---
 

package/src/orchestration/headless-observer.ts

@@ -24,16 +24,23 @@ export type { HeadlessObserverOptions };
  * rejections never become unhandled.
  */
 export function makeSafeObserver<T extends object>(observer: T): T {
-  return new Proxy(observer, {
-    get(target, prop) {
-      // Guard the ACCESS too, and resolve getters against the target (not the
-      // proxy): an observer exposing a method via an accessor that touches a
-      // private field would otherwise throw at `observer?.onX?.(...)` property
-      // access — outside the call-site try/catch below — which is exactly the
-      // failure this wrapper exists to absorb.
+  // Proxy a FRESH empty object and close over the real observer instead of
+  // proxying the observer directly: a `get` trap on the observer itself may
+  // not substitute a wrapper for a non-writable non-configurable own data
+  // property (Proxy invariant), so a frozen observer — a common defensive
+  // pattern for SDK callback bags — would make the engine throw TypeError at
+  // the property ACCESS, outside every try/catch. An empty target has no own
+  // properties, so no invariant can ever bind the trap.
+  return new Proxy({} as T, {
+    get(_target, prop) {
+      // Guard the ACCESS too, and resolve getters against the observer (not
+      // the proxy): an observer exposing a method via an accessor that touches
+      // a private field would otherwise throw at `observer?.onX?.(...)`
+      // property access — outside the call-site try/catch below — which is
+      // exactly the failure this wrapper exists to absorb.
       let value: unknown;
       try {
-        value = Reflect.get(target, prop, target);
+        value = Reflect.get(observer, prop, observer);
       } catch (err) {
         logger.warn(`[observer] Ignored error reading observer.${String(prop)}:`, err);
         return undefined;
@@ -41,7 +48,7 @@ export function makeSafeObserver<T extends object>(observer: T): T {
       if (typeof value !== 'function') return value;
       return (...args: unknown[]) => {
         try {
-          const out = value.apply(target, args);
+          const out = value.apply(observer, args);
           if (out && typeof (out as Promise<unknown>).then === 'function') {
             return (out as Promise<unknown>).catch((err: unknown) => {
               logger.warn(`[observer] Ignored rejection from observer.${String(prop)}:`, err);
@@ -54,6 +61,17 @@ export function makeSafeObserver<T extends object>(observer: T): T {
         }
       };
     },
+    has(_target, prop) {
+      try {
+        return Reflect.has(observer, prop);
+      } catch {
+        return false;
+      }
+    },
+    // Keep `instanceof` transparent (legal on an extensible target).
+    getPrototypeOf() {
+      return Reflect.getPrototypeOf(observer);
+    },
   }) as T;
 }