SemianiakaVY/openpencil
1
0
Fork 0
Code Issues Pull requests Actions 1 Packages Projects Releases Wiki Activity

Fix Kiwi encodeColor crash on colors missing alpha field

Browse source 
Defensive safeColor() guard at serialization boundary — fills, strokes,
effects, and gradient stops all go through it before Kiwi encoding.
Also fix const→let for execResult in AI adapter (reassigned by step budget).
This commit is contained in:
Anton A S 2026-03-13 11:48:43 +03:00
parent d46b4b11a4
commit b5fd31b557
2 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
packages/core/src/kiwi-serialize.ts
View file

@ -251,16 +251,20 @@ function exportTextData(node: SceneNode): NodeChange['textData'] {
}
}
function safeColor(c: { r: number; g: number; b: number; a?: number }) {
return c.a !== undefined ? c : { r: c.r, g: c.g, b: c.b, a: 1 }
}
function fillToKiwiPaint(f: SceneNode['fills'][number]): Paint {
const paint: Paint = {
type: f.type,
color: f.color,
color: safeColor(f.color),
opacity: f.opacity,
visible: f.visible,
blendMode: f.blendMode ?? 'NORMAL'
}
if (f.gradientStops) {
paint.stops = f.gradientStops.map((s) => ({ color: s.color, position: s.position }))
paint.stops = f.gradientStops.map((s) => ({ color: safeColor(s.color), position: s.position }))
}
if (f.gradientTransform) paint.transform = f.gradientTransform
if (f.imageHash) paint.image = { hash: f.imageHash }
@ -406,7 +410,7 @@ export function sceneNodeToKiwi(
const fillPaints = node.fills.map(fillToKiwiPaint)
const strokePaints = node.strokes.map((s) => ({
type: 'SOLID' as const,
color: s.color,
color: safeColor(s.color),
opacity: s.opacity,
visible: s.visible,
blendMode: 'NORMAL' as const
@ -442,7 +446,7 @@ export function sceneNodeToKiwi(
if (node.effects.length > 0) {
nc.effects = node.effects.map((e) => ({
type: e.type === 'LAYER_BLUR' ? 'FOREGROUND_BLUR' : e.type,
color: e.color,
color: safeColor(e.color),
offset: e.offset,
radius: e.radius,
spread: e.spread,

2
packages/core/src/tools/ai-adapter.ts
View file

@ -164,7 +164,7 @@ export function toolsToAI(
options.onBeforeExecute?.(def)
try {
const execResult = await def.execute(options.getFigma(), args)
let execResult = await def.execute(options.getFigma(), args)
if (def.mutates && options.onFlashNodes) {
const ids = extractNodeIds(execResult)
if (ids.length > 0) options.onFlashNodes(ids)