Refresh manage-zerocopy-sapbdc with v1.1.0 audit pass

[jdanielmyers]

Summary

Runs the existing manage-zerocopy-sapbdc skill through the v1.1.0 audit pipeline. The holistic rewriter applied:

• Audit-driven mechanical fixes (1 blocking + 7 advisory checks addressed in the previous round)
• A builder-audience pass that strips any internal-Snowflake framing while preserving the SAP BDC technical content (which is built on a public feature)
• README.md updated to reference Snowflake-Labs repo instead of internal Snowflake-Solutions

Test plan

• audience.no_internal_refs passes on the new SKILL.md
• check_residual_blocking.py exits 0 against the staged version
• Frontmatter matches Labs conventions (all required fields present, tools list format, proper quoting)
• No nested SKILL.md in sub-skill directories
• No Snowflake-Solutions references remaining
• LICENSE present
• YAML frontmatter parses cleanly

Post-rewrite residual

• 0 blocking failures
• 2 advisory: tdd.body.has_when_to_use and tdd.body.word_budget (soft style guidelines, acceptable)

[jdanielmyers]

Audit Report: manage-zerocopy-sapbdc

Re-audit (post-v1.1.1 + critical fixes). Original report superseded.

Summary

Category	🔴	🟡	🟢
Frontmatter	0	0	0
Structure	0	1	1
Workflow	0	0	1

Critical 🔴

None — all critical findings from the original audit have been addressed.

Warnings 🟡

1. Sub-flow publish/INSTRUCTIONS.md is 519 lines (exceeds 500-line guideline) → Consider extracting the CSN generation section to its own file (a csn-generator/ sub-flow already exists, so verify it's being leveraged to reduce publish/ size).

Suggestions 🟢

1. The router SKILL.md (134 lines) is well-structured — the menu in Step 1 is clear, stopping points are present, and sub-flows are well-documented.
2. Consider adding a demo-url value to the frontmatter (currently empty string).

Verdict

Ready to merge — Router is clean, stopping points are correct, sub-flows exist and are referenced properly. The 519-line sub-flow is a minor concern but doesn't block merge.