Skip to content

Fix inaccuracies found during end-to-end validation on dev - #3

Merged
AlejandroEsquivel merged 2 commits into
mainfrom
fix/skill-validation-fixes
Oct 1, 2026
Merged

AlejandroEsquivel merged 2 commits into
mainfrom
fix/skill-validation-fixes

Conversation

@cubeorgdev

@cubeorgdev cubeorgdev Bot commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Five surgical doc fixes to the blocks-api skill, found by installing it via install.sh and following it step by step against api.dev.blocks.team with a real session.

  • 401/403 rows: a malformed Authorization header (e.g. Bearer <key>) returns 403, not 401. 401 only happens when the header is missing.
  • Tool-call payload: args are top-level keys next to __name__, not nested under "input". Now documented loosely as {"__name__": "<tool>", ...tool-specific args} so the skill doesn't pin a per-tool format.
  • Plan vs transcript: the plan is written before implementation, so step 3 now notes that fixes, PRs, and outcomes live in the transcript. Prevents stopping after step 2.
  • Redundant call: step 2 re-fetched the session to read plan_artifact_ids. Step 1 now saves the response and step 2 reads it.
  • Typo: "working with the Blocks" → "working with Blocks".

Verification

  • Edited step 1 and 2 snippets run as written against dev: artifact resolved, plan downloaded with HTTP 200.
  • CI "Validate skills" checks run locally: secret/internal-host scan clean, install.sh parses.

🤖 Generated with Claude Code

- 401/403: a malformed Authorization header returns 403, not 401
- Tool-call payload: args are top-level keys, not nested under "input"
- Note that outcomes live in the transcript, not the pre-implementation plan
- Step 2 reuses the session response from step 1 instead of refetching
- Typo in description: "working with the Blocks" -> "working with Blocks"

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@cubeorgdev

cubeorgdev Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor Author

This pull request was created from a Blocks session.

View in dashboard | View on desktop

@AlejandroEsquivel
AlejandroEsquivel marked this pull request as ready for review October 1, 2026 22:51
Comment thread skills/blocks-api/references/sessions-get-overview.md Outdated
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@AlejandroEsquivel
AlejandroEsquivel merged commit 8a43337 into main Oct 1, 2026
4 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