Skip to content

Promote openflow-spcs-privatelink from Snowflake-Solutions#30

Merged
jdanielmyers merged 1 commit into
mainfrom
promote/openflow-spcs-privatelink
May 21, 2026
Merged

Promote openflow-spcs-privatelink from Snowflake-Solutions#30
jdanielmyers merged 1 commit into
mainfrom
promote/openflow-spcs-privatelink

Conversation

@jdanielmyers
Copy link
Copy Markdown
Member

Summary

Promotes the openflow-spcs-privatelink 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
  • Sub-skills folded into <sub>/INSTRUCTIONS.md per the umbrella precedent (manage-zerocopy-sapbdc)

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
  • No nested SKILL.md in sub-skill dirs (all converted to INSTRUCTIONS.md)

@jdanielmyers
Copy link
Copy Markdown
Member Author

jdanielmyers commented May 20, 2026

Audit Report: openflow-spcs-privatelink

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

Summary

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

Critical 🔴

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

Warnings 🟡

  1. Frontmatter name is setup-openflow-privatelink but the skill folder is openflow-spcs-privatelink → The name field should match the folder name for consistent discovery and invocation. Either rename the folder to setup-openflow-privatelink or change the frontmatter name to openflow-spcs-privatelink.

Suggestions 🟢

  1. Excellent single-file skill: clear architecture diagram (ASCII), well-structured 2-phase workflow (AWS infra → Snowflake side), stopping points at health-check gates, troubleshooting quick-reference table, and ordered cleanup instructions.

Verdict

Needs minor fixes — The name/folder mismatch should be resolved before merge. Everything else is clean and well-structured.

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/openflow-spcs-privatelink branch from 479359f to 090980e Compare May 20, 2026 21:13
@jdanielmyers jdanielmyers merged commit 79403ba into main May 21, 2026
1 check passed
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