Skip to content

docs(skill): avoid blind context show when DATAMODEL missing#57

Merged
eddietejeda merged 2 commits intomainfrom
fix/skill-context-show-when-missing
Apr 24, 2026
Merged

docs(skill): avoid blind context show when DATAMODEL missing#57
eddietejeda merged 2 commits intomainfrom
fix/skill-context-show-when-missing

Conversation

@eddietejeda
Copy link
Copy Markdown
Contributor

Problem

Agents following the skill often run hotdata context show DATAMODEL immediately (e.g. on /hotdata). When no context exists, the CLI exits 1; parallel tool runs can cancel siblings.

Change

  • Always context list first; only show DATAMODEL if the stem appears.
  • Document that missing context is normal for a new workspace—not a hard failure; avoid speculative show in parallel with other shell tools.
  • Align WORKFLOWS Model init, MODEL_BUILD, and DATA_MODEL.template storage note.

Branch

fix/skill-context-show-when-missing from main.

- Call out exit 1 as normal when DATAMODEL missing; warn on parallel bash
- SKILL: agents blockquote, steps, Model workflow, sandbox promote, query step 0, context list/show bullets
- WORKFLOWS, MODEL_BUILD, DATA_MODEL.template: align initialize/read paths
@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

claude[bot]
claude Bot previously approved these changes Apr 24, 2026
Resolve skill doc conflicts: keep context:<STEM> notation + list-before-show,
analysis vs context:DATAMODEL, and merged workflow/sandbox/query steps.
@eddietejeda eddietejeda merged commit dd97968 into main Apr 24, 2026
9 checks passed
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