Skip to content

docs(plugin): polish workspace-load skill guidance after #692 - #698

Merged
padak merged 2 commits into
mainfrom
docs/kbagent-skill-workspace-load-polish
Aug 25, 2026
Merged

docs(plugin): polish workspace-load skill guidance after #692#698
padak merged 2 commits into
mainfrom
docs/kbagent-skill-workspace-load-polish

Conversation

@padak

@padak padak commented Aug 25, 2026

Copy link
Copy Markdown
Member

Docs-only follow-up to #692 (plugin ships via git, no release needed). Improvements beyond the mandatory sync:

  • SKILL.md triggers: added the phrases a user would actually say for the new behavior (zero-copy clone, workspace load type, table too big to load, load table into workspace cheaply).
  • workspace-workflow.md: cheapest-first ordering hint (skip load via read-only storage > clone > view > copy), a verify-what-happened note pointing at the per-table tables[].load_type report, and a correction of a misleading example shipped in feat(workspace): zero-copy CLONE by default, large-COPY guard, fire-and-forget timeout fix #692: the size guard only fires for a resolved COPY, so --force on an explicit --load-type clone is a no-op (and clone never needs a longer --timeout). Replaced with two accurate examples and clarified that --json mode never prompts -- --force must be passed up front.
  • typify-table-workflow.md / storage-types-workflow.md: the tables these workflows load are exactly the ones likely to exceed 1 GiB; added a short note about the auto-clone default and the non-interactive --force requirement.
  • keboola-expert.md: agents parsing failures need to know STORAGE_JOB_TIMEOUT is now exit 4 (retryable) and the job keeps running server-side. 52,086 B of the 70,000 B budget.

Rejected (checked, no change needed): gotchas.md #692 entry already matches the file's best entries; the 60 s STORAGE_JOB_MAX_WAIT claim in gotchas.md is still literally true (workspace load uses its own constant); semantic-layer/stream workflows load small targeted tables; sql-migration-workflow does not call workspace load at all.

Checks: make skill-check OK, check_command_sync.py OK (267 commands), test_agent_prompt.py 42 passed.


Open in Devin Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

padak added 2 commits August 25, 2026 15:28
- SKILL.md triggers: zero-copy clone, workspace load type, table too big
  to load, load table into workspace cheaply
- workspace-workflow: cheapest-first ordering (skip load > clone > view >
  copy), verify-via-tables[] note, and fix a misleading example -- the
  size guard only fires for resolved COPY, so --force on an explicit
  clone is a no-op; --json mode never prompts (--force up front)
- typify-table / storage-types workflows: warn that a large typeless
  table COPY needs --force in non-interactive mode
- keboola-expert: STORAGE_JOB_TIMEOUT is now exit 4 (retryable) and the
  job keeps running server-side
…sktop limit

test_skill_frontmatter caps the frontmatter description at 1024 chars
(issue #447); the new workspace-load triggers pushed it to 1088. Keep
the two strongest new triggers (zero-copy clone, workspace load type)
and drop the descriptive '(backup/restore)' parenthetical.
@padak
padak force-pushed the docs/kbagent-skill-workspace-load-polish branch from 58d0749 to 735d5bf Compare August 25, 2026 13:28
@padak
padak merged commit b5d4be3 into main Aug 25, 2026
3 checks passed
@padak
padak deleted the docs/kbagent-skill-workspace-load-polish branch August 25, 2026 13:49
@padak padak mentioned this pull request Aug 25, 2026
13 tasks
padak added a commit that referenced this pull request Aug 25, 2026
Bump pyproject to 0.91.0, add the changelog entry covering every PR merged since v0.90.1 (#627, #681, #691, #692, #693, #694, #695, #696, #697, #698), resolve all 54 vNEXT version-gate placeholders, and run version-sync + skill-gen.

No web/frontend changes in this batch, so no whatsnew.ts entry.
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