Repository navigation
chore(release): 0.95.0 - #774
martinsifra wants to merge 1 commit into
Conversation
Covers everything merged since v0.94.0: #768 (uv audit / uv >= 0.12.15), #772 (device-login "press c to copy") and #773 (test housekeeping). No `vNEXT` placeholders were outstanding (`make vnext-check` clean), and no `whatsnew.ts` entry is added -- nothing under `web/` changed this release, so the popup correctly has nothing new to show. gotchas.md: #772 reordered the device-login panel without updating the doc surface. The 0.92.0 "one-click link is conditional" gotcha quoted the old label ("Or open this link (code pre-filled):") and the old ordering, so a relay keying on either now picks the wrong line -- rewritten, plus a new entry for `press c to copy` being absent in exactly the situations an agent drives (`--json`, no TTY, no clipboard command). Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
7703054 to
df2cb0f
Compare
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 3/5) · profile keboola-mcp-server
Release PR is mechanically sound but carries scope creep and an admittedly red build, so a human release owner should confirm before tagging.
Impact flags: scope creep · possible rollback re-introduction — see Check Run summary.
Concerns:
scratchpad/zajca-round2-replies.md: Unposted PR-review reply drafts committed; unrelated to this release's scopetasks/ship-DMD-1900.md: AI working-note/planning files committed as repo content (6 tasks/* files)pyproject.toml: Release PR to be tagged/published while build is admittedly red pending #773
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 2/5) · profile keboola-mcp-server
Release PR 0.95.0 has clean version/changelog mechanics but ships six accidental dev-note files that contradict its declared scope.
Impact flags: scope creep — see Check Run summary.
Concerns:
scratchpad/zajca-round2-replies.md: Unposted review-reply drafts committed to repo; internal notes, not release materialtasks/ship-DMD-1900.md: Merge-request dev notes committed; outside declared release scope (#772/#768/#773)tasks/pr-703-review.md: PR-703 review write-up committed; contradicts body's 'nothing else' scope claim
Suggested reviewers: keboola/ai-swimlane-kai-assistant
|
Closing unreleased. The intent was to ship #736 (the What this PR would have released instead is #772 ( One item does need to be carried forward rather than discarded — see the note on the branch below. 🤖 Generated with Claude Code |
Release PR for 0.95.0 — batches everything merged since
v0.94.0.Rebased onto
mainnow that #773 has landed.make checkexit 0 (6798 passed, 12 skipped),make vnext-check— all 355 gates resolve.Scope — the three PRs the tag will contain
feat(auth): "press c to copy" for the device-login linkchore(deps): drop pip-audit, gate CI onuv audit, require uv >= 0.12.15 (CLI-16)fix(test): un-redmain— type-clean and machine-independent_url_copytestsgit log v0.94.0..mainconfirms nothing else.Checklist
pyproject.toml→0.95.0(minor: feat(auth): add "press c to copy" for the device-login link #772 is afeat)Note, matching how 0.94.0 covered its chore PRsmake vnext-resolve+make vnext-check— no placeholders were outstanding, nothing rewrittenmake version-sync→plugin.json,marketplace.json,uv.lockmake skill-gen— idempotent, no command changes this releasewhatsnew.ts— deliberately not added:git diff v0.94.0..main -- web/is empty, so the release ships nothing UI-visible and the popup correctly has nothing newkeboola-expert.md,CLAUDE.md,commands-reference.md— reviewed, no drift (no command added/removed/renamed; these describe device-login behaviour, not panel text)gotchas.md— needed fixing, see belowmake checkexit 0make test-e2e— not run; needsE2E_API_TOKEN/E2E_URLgotchas.md: #772 changed the panel and left the docs behind
#772 reordered the device-login panel without touching any doc surface. The 0.92.0 gotcha "the one-click link is conditional" quoted the old label (
"Or open this link (code pre-filled):") and the old ordering — the pre-filled link used to be last, it is now the headline and the type-it-by-hand URL is the fallback. An agent relaying the panel by keying on that label, or assuming the first URL is the manual one, now picks the wrong line. Rewritten, with the reshuffle taggedv0.95.0.Added a second entry for
press c to copy, because it is absent in exactly the situations an agent operates in —--json, no TTY (a background shell is not one), or no native clipboard command — so an agent must relay the link itself and never tell the user to pressc.After merge
Tag the merge commit — that push is the release:
Re-check the scope against that commit first (checklist step 15): anything merged between now and then needs a changelog bullet, since
changelog-checkonly proves each release has an entry, never that an entry covers every commit under the tag.🤖 Generated with Claude Code