Skip to content

feat: add create_data_element MCP tool#21

Closed
connor-savage wants to merge 1 commit intobot/mainfrom
factory/create_data_element
Closed

feat: add create_data_element MCP tool#21
connor-savage wants to merge 1 commit intobot/mainfrom
factory/create_data_element

Conversation

@connor-savage
Copy link

Summary

Add create_data_element MCP tool to Chip.

Generated by chip-factory pipeline.

Generated by chip-factory pipeline.
@connor-savage connor-savage requested a review from a team as a code owner February 26, 2026 06:00
@svc-snyk-github-jira
Copy link

svc-snyk-github-jira commented Feb 26, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@connor-savage
Copy link
Author


⏺ Full pipeline succeeded for create_data_element:

  ┌────────────────────────────────────────┬────────────────────────────────────────────┐
  │                 Check                  │                   Status                   │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ Compilation                            │ Clean build                                │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ Lint                                   │ Passed (fixed 2 errcheck warnings in test) │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ Unit tests                             │ 7/7 pass                                   │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ MCP: create with required fields       │ Created asset successfully                 │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ MCP: create with optional display_name │ Created asset successfully                 │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ MCP: empty name validation             │ Correctly returned error                   │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ MCP: empty domain_id validation        │ Correctly returned error                   │
  ├────────────────────────────────────────┼────────────────────────────────────────────┤
  │ PR                                     │ https://github.com/collibra/chip/pull/21   │
  └────────────────────────────────────────┴────────────────────────────────────────────┘

  The lint gate worked — Claude found and fixed 2 errcheck warnings before open-pr ran. Cost $0.92, 6m33s total

@connor-savage connor-savage deleted the factory/create_data_element branch February 26, 2026 15:27
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