Skip to content

Promote dynamic-tables-guidance from Snowflake-Solutions#19

Draft
jdanielmyers wants to merge 1 commit into
mainfrom
promote/dynamic-tables-guidance
Draft

Promote dynamic-tables-guidance from Snowflake-Solutions#19
jdanielmyers wants to merge 1 commit into
mainfrom
promote/dynamic-tables-guidance

Conversation

@jdanielmyers
Copy link
Copy Markdown
Member

Summary

Promotes the dynamic-tables-guidance skill from the internal Snowflake-Solutions/cortex-code-skills repo to public Labs, rewritten for an external builder audience (developers, data engineers).

Run through the v1.1.0 audit pipeline:

  • Holistic LLM rewrite for builder audience
  • All internal Snowflake references substituted with public equivalents (e.g., Snowhouse -> ACCOUNT_USAGE)
  • audience.no_internal_refs check passes

Test plan

  • Audit blocking_failures == 0
  • audience.no_internal_refs passes
  • Frontmatter matches Labs conventions (snowflake-docs / manage-zerocopy-sapbdc precedents)
  • LICENSE present (Snowflake)
  • No Snowflake-Solutions repo path leakage

@jdanielmyers
Copy link
Copy Markdown
Member Author

jdanielmyers commented May 20, 2026

Audit Report: dynamic-tables-guidance

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

Summary

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

Critical 🔴

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

Warnings 🟡

None.

Suggestions 🟢

  1. The combination of a quick decision flowchart, comparison matrix, pipeline pattern with annotated SQL, monitoring queries, production readiness checklist, and hybrid DT+Task pattern makes this a very complete guidance skill at 173 lines.

Verdict

Ready to merge — Clean frontmatter, well-structured guidance with two explicit stopping points before destructive operations (CREATE OR REPLACE and ALTER ... SUSPEND/DROP), comprehensive Common Mistakes section, and references to sub-files. No issues.

Re-staged with v1.1.1 holistic prompt that adds stopping-point markers,
correct INSTRUCTIONS.md sub-flow cross-refs, and drops invalid tool
snowflake_object_search.
@jdanielmyers jdanielmyers force-pushed the promote/dynamic-tables-guidance branch from 622bc38 to 0d713dc Compare May 20, 2026 20:54
@sfc-gh-kmuir
Copy link
Copy Markdown
Collaborator

Still in draft mode from last month, will the updated DT bundled skill be more comprehensive?

@sfc-gh-kmuir sfc-gh-kmuir marked this pull request as draft May 22, 2026 22:28
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.

2 participants