Skip to content

Promote entity-resolution from Snowflake-Solutions#17

Merged
jdanielmyers merged 1 commit into
mainfrom
promote/entity-resolution
May 21, 2026
Merged

Promote entity-resolution from Snowflake-Solutions#17
jdanielmyers merged 1 commit into
mainfrom
promote/entity-resolution

Conversation

@jdanielmyers
Copy link
Copy Markdown
Member

Summary

Promotes the entity-resolution 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: entity-resolution

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. At 198 lines, this skill packs an impressive amount of domain knowledge (multi-path matching, contrastive embeddings, benchmark mode) while staying well under the 500-line limit. Excellent density.
  2. The references/ directory structure is well-organized with templates for each concern (normalization, blocking, matching, cost estimation, etc.).

Verdict

Ready to merge — Clean frontmatter, well-structured multi-path workflow with 10 explicit stopping points, domain profiles, benchmark mode, and comprehensive "Common Mistakes" section. No issues.

@jdanielmyers jdanielmyers force-pushed the promote/entity-resolution branch from 4b6d084 to 835d47e Compare May 20, 2026 20:52
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/entity-resolution branch from 835d47e to 0e2273a Compare May 21, 2026 01:44
@jdanielmyers jdanielmyers merged commit 314cef2 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