Skip to content

chore(copilot): sync mothership contracts - #7316

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/ttl-and-timezone-fixes
Aug 31, 2026
Merged

chore(copilot): sync mothership contracts#7316
waleedlatif1 merged 1 commit into
stagingfrom
fix/ttl-and-timezone-fixes

Conversation

@j15z

@j15z j15z commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • sync generated table-tool contracts with Mothership's Expiration/TTL contract
  • refresh canonical metrics and trace-attribute mirrors found by the full contract sync

Type of Change

  • Bug fix

Testing

  • bun run lint
  • bun run check:audits
  • block registry audit against origin/staging
  • all nine Mothership contract freshness checks using the explicit sibling-repository path

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Generated outputs are current and checks pass
  • No new warnings introduced
  • I confirm this PR targets staging

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Aug 31, 2026 11:25pm UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.

Re-trigger cubic

@j15z
j15z force-pushed the fix/ttl-and-timezone-fixes branch from dca9e93 to e042278 Compare August 31, 2026 23:25
@greptile-apps

greptile-apps Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR synchronizes generated Copilot contracts with Mothership’s TTL semantics and canonical telemetry identifiers.

  • Clarifies expiration behavior across insert, update, filtered update, and batch table operations.
  • Adds three canonical Copilot metrics.
  • Adds four canonical subagent trace attributes.

Confidence Score: 5/5

The PR appears safe to merge with no concrete correctness, security, or contract-consistency defects identified.

The generated catalog and runtime-schema descriptions remain synchronized, the documented TTL behavior matches the underlying mutation semantics, and telemetry identifiers are added consistently to their maps and value lists.

Important Files Changed

Filename Overview
apps/sim/lib/copilot/generated/metrics-v1.ts Adds three metric identifiers consistently to both the typed metric map and sorted value list.
apps/sim/lib/copilot/generated/tool-catalog-v1.ts Clarifies TTL omission and null semantics across generated table-tool catalog descriptions.
apps/sim/lib/copilot/generated/tool-schemas-v1.ts Mirrors the updated TTL descriptions in the generated runtime schemas.
apps/sim/lib/copilot/generated/trace-attributes-v1.ts Adds four subagent trace identifiers consistently to the typed map and sorted value list.

Reviews (1): Last reviewed commit: dca9e93 | Re-trigger Greptile

@waleedlatif1
waleedlatif1 merged commit 8309ad5 into staging Aug 31, 2026
26 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/ttl-and-timezone-fixes branch August 31, 2026 23:47
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