ByoungSooPark/modeling-app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: mutateKwArg was appending not overwriting args

Browse Source
This commit is contained in:
Adam Chalmers
2025-01-23 16:34:30 -06:00
committed by Nick Cameron
parent e8961d4190
commit 3cecb99aa1
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
e2e/playwright/sketch-tests.spec.ts
View File

@ -262,7 +262,7 @@ test.describe('Sketch tests', () => {
await expect(page.getByTestId('segment-overlay')).toHaveCount(2) await expect(page.getByTestId('segment-overlay')).toHaveCount(2)
// drag startProfieAt handle // drag startProfileAt handle
await page.mouse.move(startPX[0], startPX[1]) await page.mouse.move(startPX[0], startPX[1])
await page.mouse.down() await page.mouse.down()
await page.mouse.move(startPX[0] + dragPX, startPX[1] - dragPX, step5) await page.mouse.move(startPX[0] + dragPX, startPX[1] - dragPX, step5)

5
src/lang/modifyAst.ts
View File

@ -212,12 +212,13 @@ export function mutateKwArg(
node: CallExpressionKw, node: CallExpressionKw,
val: Expr val: Expr
): boolean { ): boolean {
node.arguments.forEach((arg, i) => { for (let i = 0; i < node.arguments.length; i++) {
const arg = node.arguments[i]
if (arg.label.name === label) { if (arg.label.name === label) {
node.arguments[i].arg = val node.arguments[i].arg = val
return true return true
} }
}) }
node.arguments.push(createLabeledArg(label, val)) node.arguments.push(createLabeledArg(label, val))
return false return false
} }