Skip to content

Promote ml-migration from Snowflake-Solutions#24

Draft
jdanielmyers wants to merge 1 commit into
mainfrom
promote/ml-migration
Draft

Promote ml-migration from Snowflake-Solutions#24
jdanielmyers wants to merge 1 commit into
mainfrom
promote/ml-migration

Conversation

@jdanielmyers
Copy link
Copy Markdown
Member

Summary

Promotes the ml-migration 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: ml-migration

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

Summary

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

Critical 🔴

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

Warnings 🟡

None.

Suggestions 🟢

  1. The "Red Flags" section with refused rationalizations is excellent for preventing agent shortcutting. The migration-config.yaml pattern provides good file-based state tracking. 5 explicit stopping points cover all critical decision junctures.
  2. The error handling table at the bottom is a useful quick-reference for common failures.

Verdict

Ready to merge — Well-structured dual-path skill (Inference I1-I8, Training T1-T9) with comprehensive stopping points, refused rationalizations, and platform detection logic.

@jdanielmyers jdanielmyers force-pushed the promote/ml-migration branch from 93145ac to 7d46c49 Compare May 20, 2026 21:09
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.

Note: tdd.discipline.has_red_flags residual (pre-existing, not v1.1.1 regression).
@jdanielmyers jdanielmyers force-pushed the promote/ml-migration branch from 7d46c49 to 0196c00 Compare May 20, 2026 21:58
@sfc-gh-kmuir sfc-gh-kmuir marked this pull request as draft May 22, 2026 22:22
@sfc-gh-kmuir
Copy link
Copy Markdown
Collaborator

There is a more updated v2 version available in Snowflake Solutions -- in staging mode! when it reached production, it will be ready to be ported over

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