vibe-and-thrive 1.8.3 → 1.8.5

package/.claude/commands/idea.md

@@ -115,17 +115,11 @@ Once the user confirms, write the idea file:
    - Any unresolved decisions
    ```
 
-3. Open the file for review:
-   ```bash
-   open -a TextEdit docs/ideas/{feature-name}.md
-   ```
-
-4. Say: "I've written the idea to `docs/ideas/{feature-name}.md` and opened it in TextEdit.
+3. Say: "I've written the idea to `docs/ideas/{feature-name}.md`.
 
    Review it and let me know:
    - **'approved'** - Ready to split into PRDs
-   - **'edit [changes]'** - Tell me what to change
-   - Or make edits in the file and say **'done'**"
+   - **'edit [changes]'** - Tell me what to change"
 
 **STOP and wait for user response. Do not proceed until they say 'approved' or 'done'.**
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vibe-and-thrive",
-  "version": "1.8.3",
+  "version": "1.8.5",
   "description": "Ship quality code faster with AI - RALPH autonomous loop, Claude Code hooks, and pre-commit checks",
   "author": "Allie Jones <allie@allthrive.ai>",
   "license": "MIT",

package/ralph/browser-verify/verify.ts

@@ -273,17 +273,8 @@ async function verify(options: VerifyOptions): Promise<VerifyResult> {
 
       // Capture actual console.error calls
       if (type === 'error') {
-        // Ignore common noise (favicons, devtools, resource loading 404s, sourcemaps)
-        const ignoredPatterns = [
-          'favicon',
-          'DevTools',
-          'Failed to load resource',  // Browser's generic 404/network error message
-          '.map',                      // Sourcemap 404s
-          'net::ERR_',                 // Network errors
-          'the server responded with a status of 4',  // 4xx errors
-        ];
-        const shouldIgnore = ignoredPatterns.some(pattern => text.includes(pattern));
-        if (!shouldIgnore) {
+        // Only ignore truly benign noise (favicons, devtools)
+        if (!text.includes('favicon') && !text.includes('DevTools')) {
           consoleErrors.push(text);
         }
       }
@@ -392,15 +383,23 @@ async function verify(options: VerifyOptions): Promise<VerifyResult> {
     // Check network errors
     result.networkErrors = networkErrors;
     if (options.checkNetwork && networkErrors.length > 0) {
-      // Only fail on actual errors, not 404s for optional resources
+      // Filter out truly benign 404s (static resources)
       const criticalErrors = networkErrors.filter(e =>
         !e.includes('favicon') &&
         !e.includes('.map') &&
-        !e.includes('hot-update')
+        !e.includes('hot-update') &&
+        !e.includes('.woff') &&
+        !e.includes('.ttf') &&
+        !e.includes('.png') &&
+        !e.includes('.jpg') &&
+        !e.includes('.svg') &&
+        !e.includes('.ico')
       );
       if (criticalErrors.length > 0) {
-        result.warnings.push(`${criticalErrors.length} network error(s) detected`);
-        // Don't fail on network errors by default, just warn
+        // Show actual URLs so Claude knows what to fix
+        result.errors.push(`Network errors (fix these):`);
+        criticalErrors.slice(0, 5).forEach(e => result.errors.push(`  ${e}`));
+        result.pass = false;
       }
     }