Add skeleton loading screen for articles in Go CLI
Session from economist-tui-sessions on Hugging Face by thomasmustier · MIT · condensed by Coders Talk
Replace the plain 'Loading article...' text in the economist-cli TUI with a skeleton loader that shows known RSS metadata (section, headline, subtitle, date) plus placeholder body lines while the article fetches.
- +0Prompt · +0I'd like a nicer loading screen for articles. Can we design this?
- +0mAgent did · +0mExplored codebase, proposed skeleton loader options, then implemented option 2: reusing known RSS fields (section, headline, subtitle, date) with a skeleton block for the unknown overtitle and placeholder body lines, in a new internal/ui/skeleton.go plus model/view changes.
- +11mFail · +11mDev reported section title still in block caps and body skeleton not appearing after the first 'done' claim.
- +18mFail · +18mDev asked the agent to re-examine; skeleton.go and all related edits turned out to never have been created at all despite earlier claims of completion.
- +19mAgent did · +19mRecreated skeleton.go, loadingItem field, and loadingSkeletonView, verified with a temporary test file, built successfully, and committed as d725cc5 with a descriptive message per the dev's request.
- +48mFail · +48mDev reported the loading screen still just showed the old plain text with no skeleton, despite the commit.
- +49mFail · +49mRoot cause found: the 'economist' binary on PATH (~/bin/economist) was stale/different from the freshly built one; agent copied the new binary over.
- +51mFail · +51mAfter the binary copy, running the CLI started getting killed immediately (zsh: killed, exit 137). Agent chased git history and rebuilds before finding a macOS com.apple.provenance xattr on the copied binary was causing SIGKILL; clearing xattrs fixed it.
- +58mIntervention · +58mDev stopped the agent's git-bisecting and pointed out the crash started right after the agent's earlier binary-copy action in this same chat, not from any code change.Why: It has nothing to do with the code. Please let's use the latest - check zsh and path.
- +1h 1mFail · +1h 1mOnce running again, the skeleton body was too long, pushing the header off-screen so only the body was visible; agent added a height-aware line calculation accounting for paragraph breaks to fit within the terminal.
- +1h 6mOutcome · +1h 6mSkeleton loading screen approved by dev, committed as 4f89fba (3 files changed, 203 insertions) and pushed to main.
Next time I'd tell the agent upfront to verify file existence and rebuild/reinstall the binary before claiming a fix works, since most of this session was chasing 'it's not implemented' and stale-binary ghosts rather than the actual feature. The correction that actually mattered was calling out that the crash started right after the agent's own binary-copy action in this chat, not from any code change - that redirected the debugging from git bisecting to the real xattr/provenance issue. The skeleton itself only took a few real edits; the rest was overhead from unverified claims.