Skip to content

chore: probe the breaking-change gate (do not merge) - #16

Closed
jacekradko wants to merge 1 commit into
mainfrom
jacek/verify-break-gate
Closed

jacekradko wants to merge 1 commit into
mainfrom
jacek/verify-break-gate

Conversation

@jacekradko

@jacekradko jacekradko commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Throwaway. Makes AiRunEventBase.userId required, which is a real API break, to confirm break-check fails an unacknowledged break and that the breaking-change label clears it. Closing without merging.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@github-actions

github-actions Bot commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

API Changes Report

Generated by Break Check on 2026-09-10T12:30:19.342Z

Summary

Metric Count
Packages analyzed 1
Packages with changes 1
🔴 Breaking changes 1
🟡 Non-breaking changes 0
🟢 Additions 0

Warning
1 breaking change(s) detected - Major version bump required

🔴 Breaking changes index (1)

Every breaking change, up front. Full diffs are in the package sections below.

Package Subpath Change
@radkode/ai-kit ./telemetry AiRunEventBase.userId

@radkode/ai-kit

Current version: 0.3.1
Recommended bump: MAJOR → 0.4.0 (0.x: a breaking change ships in the minor position)

Subpath ./telemetry

🔴 Breaking Changes (1)

Changed: AiRunEventBase.userId
- userId?: string;
+ userId: string;

Breaking change in property AiRunEventBase.userId: Member is no longer optional


Report generated by Break Check

Last ran on 0c34963.

@jacekradko jacekradko added the breaking-change Intentional API break, acknowledged; turns API Check green label Sep 10, 2026
@jacekradko

Copy link
Copy Markdown
Contributor Author

Probe complete: break-check failed the unacknowledged break, and the breaking-change label re-ran it green on its own with no push. verify is red because the probe deliberately breaks typecheck, which is the probe's doing, not the gate's.

@jacekradko jacekradko closed this Sep 10, 2026
@jacekradko
jacekradko deleted the jacek/verify-break-gate branch September 10, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change Intentional API break, acknowledged; turns API Check green

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant