fix: unstick /autoclip runs and Settings key entry - #50
Merged
Merged
Conversation
|
Deployment failed for project clippyos with the following error: Learn More: https://vercel.link/3Fpeeb1 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- Agent tab: keep polling the run detail in hidden tabs; each poll ticks the background media-fetch job, and React Query pauses interval refetches by default, so a run left in another tab never advanced. - media-fetch-job: a run parked as MEDIA_FETCH with no job state now fails visibly on the first tick instead of returning idle forever. - Settings: Higgsfield status read through its own loader (the generic settings reader treated *_API_KEY/*_SECRET as encrypted and returned null, so a saved key showed Not configured); Crayo Test no longer maps to the app's 'Please sign in' message; Test is disabled while a card has unsaved text since it only ever checks the stored key. Co-Authored-By: Claude <noreply@anthropic.com>
SomeRandmGuyy
force-pushed
the
devin/1789013883-media-fetch-sandbox-reaper
branch
from
September 16, 2026 02:25
c1e1903 to
8306f92
Compare
…ry loop A tick that failed on a transient sandbox/API error only console.error'd, released the lock, and returned idle — so a recurring error retried silently until the 4-hour deadline while the run showed no new progress. Now the retry is written to the run as an observe line (attempts 1, 2, 4, 8…) and the run fails visibly after 30 consecutive transient errors. Co-Authored-By: Claude <noreply@anthropic.com>
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
MEDIA_FETCHwith no job state now fails visibly on the first tick instead of returning idle forever. Two such zombie runs (started 2026-09-09/10, emptyoutputs) sat in Waiting on prod; both cancelled by hand.console.error'd and retried silently until the 4 h deadline. They now show on the run as "Retrying (attempt N) after a transient error: …" (attempts 1, 2, 4, 8…) and the run fails after 30 consecutive transient errors (~12 min).loadHiggsfieldCreds(the generic settings reader treats*_API_KEY/*_SECRETnames as encrypted and returned null, so a freshly saved key showed Not configured). Crayo Test no longer surfaces the bareUNAUTHORIZEDcode that renders as the app's "Please sign in" message. Test is disabled while a card has unsaved text, since it only ever checks the stored key.Observed on prod (2026-09-16)
A fresh
/autoclipon a 4:59 video (3 clips) sat in "Sandbox started. Installing yt-dlp + ffmpeg…" for 15 min with ~140 polls and never got the tick's own "Installing…" line — i.e. every tick threw before reachingprogress(), and the transient path swallowed it. Vercel runtime logs are 403 on this plan, so the third change is what makes the reason visible. Run cancelled.Follow-up (not in this PR)
waitUntil+ delayed self-call) or run the ops cron more often than the Hobby plan allows.Test plan
tsc --noEmitclean on touched filesnode --test src/lib/media-fetch.test.ts8/8/autoclipon https://www.youtube.com/watch?v=vrY1THC_NQE (3 clips) and read the retry line, or watch it finish🤖 Generated with Claude Code