Skip to content

run-on-jobs: every command runs as pasted - #187

Merged
davanstrien merged 1 commit into
mainfrom
fix/catalog-commands
Nov 10, 2026
Merged

davanstrien merged 1 commit into
mainfrom
fix/catalog-commands

Conversation

@davanstrien

Copy link
Copy Markdown
Owner

The GLM-OCR command on the Run on Jobs Space showed {source_dataset} and other template text. The generator took the first hf jobs uv run in the file, which for glm-ocr.py is the card template the script fills in at run time.

Changes

  • Skip examples inside f-strings and one-line strings.
  • Use two placeholder styles only: your-username and capitalised names. Angle brackets are removed, because a shell reads them as redirects. This also fixes your-your-username on the WeVisOne command.
  • Add --max-samples 10 when the script has the option and the example does not set it, so a first run is short. The page and AGENTS.md say to remove it for a full run.
  • Fail the build when a command has brackets outside quotes.
  • Run the catalogue check on pull requests, so a bad command is found before the sync publishes it.

32 of 54 commands change. No recipe file changes.

Checks

  • python3 tools/build-catalog.py --verify-models passes.
  • The new check failed on six commands during development and passes now.
  • Every flag in the 51 commands is an option of its script, and all 51 pass bash -n.
  • No Job was run. The commands that gained --max-samples 10 are checked against the script options only.

🤖 Generated with Claude Code

The GLM-OCR command on the Space showed {source_dataset} and other
template text. The generator took the first `hf jobs uv run` in the file,
which for glm-ocr.py is the card template the script fills in at run time.

- Skip examples inside f-strings and one-line strings.
- Use two placeholder styles only: `your-username` and capitalised names.
  Angle brackets are removed, because a shell reads them as redirects.
  This also fixes `your-your-username` on the WeVisOne command.
- Add `--max-samples 10` when the script has the option and the example
  does not set it, so a first run is short.
- Fail the build when a command has brackets outside quotes, and run the
  catalogue check on pull requests.

32 of 54 commands change. No recipe file changes.

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
@davanstrien
davanstrien merged commit 7486c85 into main Nov 10, 2026
1 check passed
@davanstrien
davanstrien deleted the fix/catalog-commands branch November 10, 2026 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant