Skip to content

docs(agents): fix new-item-type agent to require command_support.yaml review and prohibit VCR cassette creation - #200

Merged
Alon Yeshurun (ayeshurun) merged 9 commits into
microsoft:mainfrom
ayeshurun:copilot/update-new-item-type-agent-md
Mar 29, 2026
Merged

docs(agents): fix new-item-type agent to require command_support.yaml review and prohibit VCR cassette creation#200
Alon Yeshurun (ayeshurun) merged 9 commits into
microsoft:mainfrom
ayeshurun:copilot/update-new-item-type-agent-md

Conversation

@ayeshurun

Copy link
Copy Markdown
Collaborator

📥 Pull Request

✨ Description of new changes

PR #85 (SnowflakeDatabase onboarding) exposed two gaps in the new-item-type.agent.md instructions: the agent skipped command_support.yaml entirely for simple items, and fabricated 14 VCR cassette recording files.

Step 10 (command_support.yaml)

  • Added sub-step 10a: review unsupported_items lists for rm, get, set, mkdir -- applies to all item types, not just those with definitions
  • Reorganized existing export/import/mv/cp guidance as sub-steps 10b-10e
  • All complexity patterns (Simple, Creation Params, OneLake Folders, Job Support) now reference Step 10
  • Verification table changed from "Has definitions" to "Always"

Step 11 (test parametrization)

  • Explicit prohibition: do not create VCR cassette files under tests/test_commands/recordings/. Only modify parametrization lists in conftest.py. Recordings are generated from live test runs.

This script benchmarks the startup performance of the CLI by measuring module import times, CLI invocation times, and heavy dependency loading. It allows comparisons against a baseline branch or tag.
…all item types, prohibit creating VCR cassettes

Co-authored-by: ayeshurun <98805507+ayeshurun@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ayeshurun/fabric-cli/sessions/c94f523e-ee18-4151-9b8f-e8cb2e27e41b
shirasassoon
shirasassoon previously approved these changes Mar 29, 2026
@ayeshurun
Alon Yeshurun (ayeshurun) merged commit 993dd53 into microsoft:main Mar 29, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants