@youdie006/prodex 0.40.0 → 0.40.1

package/dist/chatgpt-browser.js

@@ -1602,6 +1602,18 @@ async function reloadPageAndAwaitComposer(page) {
         cdp.close();
     }
 }
+/**
+ * The project a ChatGPT URL belongs to, as its `g-p-<id>` segment.
+ *
+ * The rebind check used to compare whole URLs, which also carry mode in a
+ * query string - and mode is exactly what differs across the reload this
+ * failure showed up in. Identity is the thing the check actually cares about:
+ * that the composer came back bound to THIS project rather than the one the
+ * tab came from.
+ */
+export function projectIdentity(url) {
+    return /\/g\/(g-p-[^/?#]+)/.exec(url)?.[1];
+}
 export function powerSliderPresentExpression() {
     return `Boolean(document.querySelector('[data-testid="composer-intelligence-picker-content"] [role="slider"]'))`;
 }
@@ -2524,12 +2536,25 @@ async function selectProject(cdp, options) {
         // prompt posted into the project the tab came from). A hard reload of the
         // project home rebinds the composer to THIS project before we send.
         const projectHome = await cdp.evaluate("location.href");
+        const home = projectIdentity(projectHome);
         // Polled with no delay, the first check could run before the reload had
         // committed, on the old document, whose composer and URL both still
         // matched. The stamp keeps that document from passing as the new one.
-        const rebound = await reloadAndAwaitComposer(cdp, RELOAD_SETTLE_TIMEOUT_MS, `location.href === ${JSON.stringify(projectHome)}`);
+        //
+        // Identity, not the whole URL: the href also carries mode in a query
+        // string, and comparing it byte-for-byte failed a reload that had landed
+        // exactly where it was asked to.
+        const rebound = await reloadAndAwaitComposer(cdp, RELOAD_SETTLE_TIMEOUT_MS, home ? `/\\/g\\/${home.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}(?![^/?#])/.test(location.href)` : "true");
         if (!rebound) {
-            throw new Error(`ChatGPT composer did not rebind after entering project "${options.project}"`);
+            // Say what it saw. This failure is rare and was previously reported as
+            // four words, which named neither the cause nor anything to try.
+            const landed = await cdp.evaluate("location.href").catch(() => "unknown");
+            const hasComposer = await cdp
+                .evaluate(`Boolean(document.querySelector('#prompt-textarea,[contenteditable="true"],textarea'))`)
+                .catch(() => false);
+            throw new Error(`ChatGPT composer did not rebind after entering project "${options.project}": after reloading, the tab was on ` +
+                `${projectIdentity(landed) ?? "no project"} (expected ${home ?? "that project"}) and the composer was ` +
+                `${hasComposer ? "present but the page never settled" : "still missing"}.`);
         }
     }
     const composerReady = await waitForExpressionTrue(cdp, `Boolean(document.querySelector('#prompt-textarea,[contenteditable="true"],textarea'))`, PROJECT_NAVIGATION_TIMEOUT_MS);

package/dist/cli-pro.js

@@ -1110,6 +1110,12 @@ export async function runAskProCommand(rest, io) {
     // Requiring --new-chat keeps that explicit rather than quietly turning a
     // continuation into a throwaway.
     const temporary = parsedAskPro.optionArgs.includes("--temporary");
+    const temporaryConflict = temporaryProjectConflict({
+        temporary,
+        ...(explicitProject !== undefined ? { explicitProject } : {})
+    });
+    if (temporaryConflict)
+        throw new Error(temporaryConflict);
     if (temporary && !newChat) {
         throw new Error("--temporary starts a throwaway chat, so it needs --new-chat. A temporary chat cannot be continued or recovered later.");
     }
@@ -1141,9 +1147,17 @@ export async function runAskProCommand(rest, io) {
     // fresh chat inside the project is exactly what "--new-chat + project"
     // produces, and the whole point of pinning a default project is that
     // consults stop landing in the general chat list. Only --target-url
-    // (pinned tab) and --project-new suppress it.
+    // (pinned tab), --project-new, and --temporary suppress it.
+    //
+    // --temporary suppresses rather than conflicts: a pinned project must not
+    // turn every throwaway send into an error, and attempting both is what
+    // produced "composer did not rebind after entering project" - a temporary
+    // chat is never saved, a project chat is, and entering a project leaves
+    // temporary mode.
     const selectionProject = explicitProject ??
-        (normalizedTargetUrl || selectionProjectNew !== undefined || suppressProject ? undefined : browserDefaults?.project);
+        (normalizedTargetUrl || selectionProjectNew !== undefined || suppressProject || temporary
+            ? undefined
+            : browserDefaults?.project);
     const reasoningAxisChosen = explicitProMode !== undefined || explicitEffort !== undefined;
     const selectionProMode = explicitProMode ?? (reasoningAxisChosen ? undefined : browserDefaults?.pro_mode);
     const selectionEffort = explicitEffort ?? (reasoningAxisChosen ? undefined : browserDefaults?.effort);
@@ -1796,6 +1810,19 @@ export function proSelectionVerified(selection) {
         return undefined;
     return /pro/i.test(selection.modelSlug);
 }
+/**
+ * A temporary chat is not saved; a project chat is. Entering a project leaves
+ * temporary mode, so asking for both is asking for two different things, and
+ * prodex used to attempt both and die inside the project step with "composer
+ * did not rebind". A pinned default project is suppressed instead of refused:
+ * the per-call flag is the more specific instruction.
+ */
+export function temporaryProjectConflict(input) {
+    if (!input.temporary || input.explicitProject === undefined)
+        return undefined;
+    return (`--temporary and --project cannot be combined: a temporary chat is never saved, and a chat inside a project is. ` +
+        `Drop --temporary to send into "${input.explicitProject}", or drop --project to send a throwaway chat.`);
+}
 export function browserSendBlockerFromError(error) {
     const blocker = typeof error === "object" && error !== null && "blocker" in error ? error.blocker : undefined;
     if (typeof blocker === "object" &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@youdie006/prodex",
-  "version": "0.40.0",
+  "version": "0.40.1",
   "description": "Local receipt bus for coordinating Codex execution with ChatGPT Pro/Projects consultation.",
   "author": "youdie006",
   "license": "MIT",