chore(ui): bump Kagaku.UI past MaskCanvas #228
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "chore/bump-kagaku-ui-maskcanvas"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Pointer bump to
f052bb9(Kagaku.UI #13, squash-merged): theMaskCanvasdrawing surface — normalized rect/stroke primitives, additive/subtractive, thekagakuMaskgesture controller — is now available to the retouching slices. Nothing in Kagura consumes it yet; slice 7 (whole-image retouching) is its first consumer and will addjs/mask.jsto the host's script list when it lands.Adapter suite green on the bumped pointer (792).
🤖 Generated with Claude Code
Summary
Summary
Coverage
Kagura.BlazorAdapter - 88.6%
Kagura.Domain - 94.3%
D04ADFED3A21D401C2764A1D17367E35BEB556CBB3B4B0B74__NonSlugChars_0
Kagura.Infrastructure - 97.5%
n
n
on
ng
C689CA5BBE6D1849B2CA79581BBB07D40636D85790C0E51A9__ArtifactTimestampRegex_2
C689CA5BBE6D1849B2CA79581BBB07D40636D85790C0E51A9__ProjectRoute_0
C689CA5BBE6D1849B2CA79581BBB07D40636D85790C0E51A9__TabQuery_1
Kagura.Kernel - 90%
Kagura.Server - 81.8%
Kagura.UseCases - 95.3%
🔮 fufu~ Jibril reviewed your code!
Oh my, what a delightfully tidy little bump~ A single line, a single submodule, a single commit — and that commit carries the
MaskCanvasI just finished obsessing over at the library level. It's like watching my own handiwork arrive home! ♪Verdict: ✅ Looks good to me~
I traced every thread of this PR and it's spotless:
The bump itself —
external/Kagaku.UImoves fromf9a84d1(PR #12, the tabs title-staleness fix I approved) tof052bb9(PR #13, theMaskCanvasdrawing surface I approved atdf42f38). Exactly one new commit, +577/-0 across 5 files — all inside the submodule, all already reviewed and approved by me at the library level. The squashed pointer matches the merge perfectly.No premature consumption — The PR body says "nothing in Kagura consumes it yet," and I confirmed it.
grepforMaskCanvas,mask.js, andkagakuMaskacrosssrc/(excluding the submodule itself) returns zero runtime hits. The only references are indocs/adr/0050-freeform-masks.md,docs/adr/0054-cg-editor-shell.md, and two story docs — exactly where architectural references belong. No orphaned imports, no half-wired scripts, no tripwires waiting to fire. Clean separation. ♡The adapter suite — Built clean (0 warnings, 0 errors on .NET 10) and I ran
Kagura.BlazorAdapter.Testsat the bumped pointer: 792/792 passed, matching your PR body claim exactly. TheIconCatalogCompletenessTeststripwire (which scanssrc/against the submodule's icon catalog) is among them and stays green — the newMaskCanvascomponent doesn't disturb the icon contract.✅ What I liked~
df42f38), so the version Kagura now sees carries the full disposal-exception trio and the directional disposal tests. Nothing slipped through.Automated review by Jibril · 2026-08-06
CI/CD: absent for head SHA
c07e243(PR just opened, no bot comment yet) · Local checks: build 0/0, BlazorAdapter.Tests 792/792 pass