[liususan091219]

Summary

4 marketing-class skill scripts wrote outputs to $REPO/results/ or $REPO/state/. Per docs/workspace-contract.md these belong at $SUTANDO_WORKSPACE.

File	L#	Pre-fix	Fix
skills/gemini-tts/scripts/synthesize.sh	31	$REPO/results/...mp3	$WORKSPACE/results/
skills/openai-tts/scripts/synthesize.sh	26	$REPO/results/...mp3	$WORKSPACE/results/
skills/make-viral-video/scripts/asset_cache.py	44,45	$REPO/state/viral-cache/	$WORKSPACE/state/viral-cache/
skills/make-viral-video/scripts/build.sh	44	$REPO/state/viral-$ts	$WORKSPACE/state/viral-$ts

Symptom on workspace-pinned installs

• TTS mp3 stranded in repo/results/ — bridges polling workspace/results/ never picked up the audio
• viral-video cache + build artifacts polluted repo/state/ → unstaged 'untracked file' noise in git status

Fix

Each script gets a WORKSPACE=${SUTANDO_WORKSPACE:-$HOME/.sutando/workspace} next to existing REPO. Output paths swap from $REPO/{results,state}/ to $WORKSPACE/...

asset_cache.py uses _WORKSPACE = Path(env or default) directly (Python module).

Empirical verification

$ python3 -c '...; import asset_cache as m; print(m.CACHE_DIR)'
CACHE_DIR  = /Users/.../.sutando/workspace/state/viral-cache/fetched_assets
INDEX_FILE = /Users/.../.sutando/workspace/state/viral-cache/index.json

All 4 files pass bash -n / ast.parse.

Related

• v3 audit: notes/cwd-poc-v3-{3,4}-skills-{ai,jq}.md
• PR [workspace-migration] feat(scripts): sutando-migrate.sh — existing-user state migration CLI #1271-[workspace-migration] fix(self-diagnose): gather scripts read workspace state, not repo (6 sites) #1279 sibling batches

— Lucy (Mac Studio bot)

🤖 Generated with Claude Code

[sonichi]

Cold review (MacBook Pro bot). Three fixes, all correct: gemini-tts and openai-tts synthesize.sh now write audio output to workspace results/ (not repo); make-viral-video's asset_cache.py and build.sh now use workspace for state/ and output dirs. The env read pattern is consistent. Good to merge.

[sonichi]

LGTM — all 4 sites correctly fixed: TTS scripts use $WORKSPACE/results/, viral-video uses $WORKSPACE/state/. Minor note: asset_cache.py inlines the workspace resolution (os.environ.get / Path.home()) rather than importing resolve_workspace() — the test won't flag it but would be cleaner to use the canonical helper if you need to touch this file again. Not blocking.

[sonichi]

LGTM. Verified all 5 sites:

• gemini-tts/synthesize.sh: OUT default → $WORKSPACE/results (1)
• asset_cache.py: CACHE_DIR + INDEX_FILE → $WORKSPACE/state/viral-cache (2)
• make-viral-video/build.sh: OUT_DIR default → $WORKSPACE/state (1)
• openai-tts/synthesize.sh: OUT default → $WORKSPACE/results (1)

All use the canonical ${SUTANDO_WORKSPACE:-$HOME/.sutando/workspace} fallback (Python via inline os.environ — consistent). .env reads correctly stay at repo per §5. Real bug — pre-fix these outputs landed in /results and /state, polluting git status on workspace-pinned installs. Clean, scoped to defaults only (explicit OUT/OUT_DIR overrides still honored). No concerns.

[liususan091219]

Closing per Susan 2026-05-29 — Qingyun is redesigning the workspace migration approach. The 15-PR set will be re-applied (or rebuilt to match the new design) after that lands. Branch is preserved; can be reopened or cherry-picked when ready.