@womp/spark-agent 4.4.1 → 5.0.0

package/.env.example

@@ -0,0 +1,6 @@
+OPENAI_API_KEY=
+ANTHROPIC_API_KEY=
+GEMINI_API_KEY=
+KAKAPO_URL=ws://127.0.0.1:5502
+PORT=8787
+SPARK_STORAGE_PATH=.spark/spark.sqlite

package/README.md

@@ -6,7 +6,23 @@ Spark Agent uses AI SDK 7 `UIMessage` and `UIMessageChunk` as its public chat pr
 JavaScript uses the host's executor and capabilities. Native browser execution and isolated QuickJS
 execution use the same model loop. The host owns conversations, artifacts, authorization, and connections.
 
-## Requirements
+## Requirements
+
+### Migrating to 5.0
+
+- Replace scene capability `read`/`edit` with `access(callback)`. Inspect the current draft
+  through `scene.nodes({ ids, name, type, parentId, offset, limit })`, not a `nodes` array.
+  Results default to 100, with a maximum page size of 1,000.
+- Update custom scene hosts to `SceneHost.access` and SDK adapters to `acquireAccess`.
+  `onCommitted` receives the acknowledged scene; keep host access suspended until synchronization succeeds.
+- Remove scene revision arguments from Geometry/OpenSCAD writes. Current parameters and overrides
+  are prepared inside protected access. Hosts must still enforce connection/project ownership.
+- Screenshot calls capture the current native viewport; remove target, view and camera options.
+- Scene health is explicitly requested through `scene.health()`, not automatically after each edit.
+- This release requires Kakapo SDK 0.7.0. Native and QuickJS executors now allow 100,000 host calls
+  per execution. Receipts, cancellation, timeout limits and uncertain-outcome handling remain enforced.
+
+### Runtime
 
 - Node.js 22.13 or newer
 - AI SDK 7
@@ -339,7 +355,10 @@ The standalone host uses a fresh QuickJS runtime for each JavaScript execution.
 or browser globals and receives only the composed capability APIs. Imports, filesystem access,
 network APIs, and host environment variables are not exposed. QuickJS limits memory, stack size,
 execution time, source size, host calls, and data crossing the host bridge. Browser hosts can instead
-choose native page execution, with the page-global access and interruption limitations described above.
+choose native page execution, with the page-global access and interruption limitations described above.
+Both executors allow 100,000 host calls per execution by default; the counter resets for each
+execution. QuickJS supports overriding this through `maxHostCalls`. Cancellation, timeouts,
+and the other execution limits still apply.
 
 QuickJS isolates generated code from the JavaScript host API; it is not an operating-system security
 boundary. Multi-tenant deployments must additionally isolate workers, authorize every Kakapo

package/dist/capabilities/geometry.d.ts

@@ -17,13 +17,11 @@ export type GeometryTarget = {
     instanceId?: number;
     source: string | null;
     instances: GeometryInstance[];
-    revision: string;
     diagnostics: string[];
 };
 export type GeometryWrite = {
     objectId: number;
     source: string;
-    expectedRevision: string;
 };
 export type GeometryWriteResult = {
     status: "applied" | "rejected";
@@ -31,7 +29,6 @@ export type GeometryWriteResult = {
     graphId?: number;
     instanceId?: number;
     affectedObjectIds: number[];
-    revision: string;
     diagnostics: string[];
 };
 export type WompGeometryApi = {

package/dist/capabilities/geometry.js

@@ -5,7 +5,6 @@ const write = z
     .object({
     objectId,
     source: z.string().trim().min(1),
-    expectedRevision: z.string().min(1),
 })
     .strict();
 export function createGeometryCapability(host, reference) {
@@ -15,26 +14,26 @@ export function createGeometryCapability(host, reference) {
                 name: "geometry-nodes",
                 version: "5",
                 description: "Create and edit visual Geometry Nodes graphs with adjustable parameters and animation.",
-                instructions: `Use womp.geometry for requested Geometry Nodes, parametric organic/procedural geometry, or animation.
+                instructions: `Use womp.geometry for requested Geometry Nodes, parametric organic/procedural geometry, or animation.
 Keep follow-up edits to an existing Geometry Nodes object in its graph, preserving its adjustable parameters and instances.
 Read the target after any scene edit this build needs. read(0) reads the scene root as a creation container.
 write updates an existing shared graph, attaches a new graph to an Area without one, or creates a new Area beneath the root/Group target. Use the returned objectId for follow-up edits.
-Read geometry-nodes-reference before writing source. Supply the complete FlowIR text and the exact revision returned by the latest read. JavaScript calls execute JavaScript; source is a FlowIR string, not JavaScript or JSON.
-After a stale revision rejection, read the latest graph and adapt the edit while preserving intervening user changes. Ask only when the edits conflict or remain ambiguous; never retry the old replacement with a refreshed revision token.
+Read geometry-nodes-reference before writing source. Supply the complete FlowIR text after inspecting the target. JavaScript calls execute JavaScript; source is a FlowIR string, not JavaScript or JSON.
+Inspect the current target before replacing its graph; preserve unrelated geometry and parameter overrides.
 If source is null and diagnostics are present, the existing graph cannot be edited without loss. Explain the limitation; never treat it as empty or create a replacement to bypass the refusal.
 Preserve unrelated statements, parameter names, and FlowIR node IDs from the read. Shared graph edits affect all listed instances; report the affected objects. Valid overrides migrate by parameter name and type. Removed or type-changed parameters return to graph defaults and are reported.
 Use min/max/step for naturally bounded parameters and omit ranges for free-form positions/rotations. Meaningful parameter names become editor labels. Do not invent label fields.
 SetMaterial assigns one scene material to every primitive it receives. Its Material input is a scene material ID, not a color; 0 selects the scene default material.
-Obtain that ID from the scene-editing skill inside one scene.edit, where those lookups live: materialList with materialGetName and materialGetColor to reuse an existing material, materialCreate with materialSetColor and materialSetName for a new one, or listSuperMaterials with materialImport for a library material. Prefer reusing a material that already matches the request over creating another, and do not recolor a shared material unless every object using it should change.
-A scene edit changes the scene revision, so create or import the material before reading the geometry target, then write, all within one JavaScript call. Each call has a fresh scope and cannot read an earlier call's variables.
+Obtain that ID from the scene-editing skill inside one scene.access, where those lookups live: materialList with materialGetName and materialGetColor to reuse an existing material, materialCreate with materialSetColor and materialSetName for a new one, or listSuperMaterials with materialImport for a library material. Prefer reusing a material that already matches the request over creating another, and do not recolor a shared material unless every object using it should change.
+Create or import the material inside scene.access before writing the geometry target, all within one JavaScript call. Each call has a fresh scope and cannot read an earlier call's variables.
 Positions and sizes are millimetres; Boolean Blend uses the conversion in geometry-nodes-reference. Coordinates are right-handed, +X right, +Y up, +Z forward. Rotations are degrees, intrinsic X-Z-Y.
 On compiler errors, repair the source in this same conversation and retry with a fresh read. A rejected write made no change. Committed/unknown effects remain in execution receipts even if a later call fails; inspect before retrying.
 After applying, read the returned object and use the scene screenshot API with emitImage to inspect the applied result.`,
                 declarations: WOMP_GEOMETRY_DECLARATIONS,
                 examples: [
                     "const target = await womp.geometry.read(objectId); print(target);",
-                    "const target = await womp.geometry.read(objectId); print(await womp.geometry.write({ objectId: target.objectId, source: editedFlowIR, expectedRevision: target.revision }));",
-                    "const materialId = await womp.scene.edit(scene => { const id = scene.materialCreate(); scene.materialSetName(id, 'Red'); scene.materialSetColor(id, { r: 255, g: 0, b: 0 }); return id; }, (await womp.scene.read()).revision); const target = await womp.geometry.read(0); const source = ['param Float size = 20.0 min=1 max=100 step=1', 'node box Primitive Type=Cube Position=Vec3(0.0,0.0,0.0) Rotation=Vec3(0.0,0.0,0.0) Size=size Roundness=0.0', `node painted SetMaterial Material=${materialId} Geometry=box.Geometry`, 'out Geometry=painted.Geometry'].join('\\n'); print(await womp.geometry.write({ objectId: target.objectId, source, expectedRevision: target.revision }));",
+                    "const target = await womp.geometry.read(objectId); print(await womp.geometry.write({ objectId: target.objectId, source: editedFlowIR }));",
+                    "const materialId = await womp.scene.access(scene => { const id = scene.materialCreate(); scene.materialSetName(id, 'Red'); scene.materialSetColor(id, { r: 255, g: 0, b: 0 }); return id; }); const target = await womp.geometry.read(0); const source = ['param Float size = 20.0 min=1 max=100 step=1', 'node box Primitive Type=Cube Position=Vec3(0.0,0.0,0.0) Rotation=Vec3(0.0,0.0,0.0) Size=size Roundness=0.0', `node painted SetMaterial Material=${materialId} Geometry=box.Geometry`, 'out Geometry=painted.Geometry'].join('\\n'); print(await womp.geometry.write({ objectId: target.objectId, source }));",
                 ],
             },
             {

package/dist/capabilities/openscad-instructions.d.ts

@@ -1 +1 @@
-export declare const OPENSCAD_INSTRUCTIONS = "Use womp.openscad to create or update one adjustable CAD part. Read an existing part before editing it.\nKeep follow-up edits to an existing CAD part in that part unless the user asks to change representation.\nread(0) returns the root container and a scene revision for creation. read(id) returns an existing\npart or container. Use the scene-editing skill when available to discover other scene objects.\ncreate requires a parentId (0 means the scene root); update requires the existing objectId.\nBoth methods compile the effective source and parameters, apply one undoable change, and check wall thickness.\nA stale revision requires reading again and adapting the edit while preserving intervening user changes.\nAsk the user only when the intended edits conflict or remain ambiguous; never reuse the old replacement\nwith a refreshed revision token. Existing overrides are preserved for parameters still present;\nparams supplies overrides to merge into the current values. Keep top-level parameter names stable.\nReturn complete source on every update and preserve unrelated geometry. Never create a replacement to repair a part.\nCompile errors return status rejected without applying. Creation requires validated placement bounds.\nAn update may apply with validation unavailable; that is a diagnostic, not proof that it compiles.\nAn applied part remains in the scene even if thickness fails or is unavailable. Use the returned objectId for repairs.\nWhen thickness fails, increase thin walls and fragile connections while preserving requested dimensions and openings.\nAt most four applied printability repairs are allowed per logical build/edit in this run. Respect the returned limit;\ndo not bypass it by recreating the part. Unavailable thickness is not a pass. Report unresolved issues honestly.\nBuild connected geometry and avoid features below 1.4 mm. OpenSCAD source dimensions use millimetres,\nindependently of the scene display unit. New parts are centered and grounded from compiled bounds.\nOpenSCAD is Z-up; the scene-editing API is Y-up. For an unrotated part at its creation-time scale and an identity parent transform,\na source point [x,y,z] mm maps to scene {x:y, y:z, z:-x} times the unitMultiplier returned by scene.read(), plus nodeGetPos(partId).\nAccount for the part and parent transforms when they are not identity; source coordinates are not scene coordinates.\nAfter a major change, use the scene screenshot API and emitImage to inspect the applied result.\nA committed effect can survive later failure. Inspect receipts and the existing part before retrying.\nDo not print source in the final answer unless requested; describe the result and any unresolved diagnostics.\n\nRUNTIME\n- Produce a single OpenSCAD file.\n- Do not use imports, includes, external libraries, text, hull, minkowski, polyhedron, or extrudes.\n- Prefer cube, sphere, cylinder, translate, rotate, color, union, difference, and intersection.\n- Do not use $fn, $fa, or $fs; tessellation controls are ignored.\n- Build faceted geometry by combining supported primitives with CSG operations and transforms.\n- Do not use scale(); size primitives directly instead.\n- Use color(\"#rrggbb\") around distinct visible parts.\n- Keep the model centered near the origin and use millimeter-like units consistently.\n- For organic blending, use goop(radius_mm) { ... }. The radius is in millimetres before parent scaling, matching scene-editing; do not multiply by project unitMultiplier. It is ordered: the first child seeds with blend 0, and each later child uses radius_mm.\n- Put core primitives first. Avoid wrapping goop children in union() when they should blend individually.\n\nSCREWS AND THREADS\n- Copy helical_ridge below for rounded screw threads, necks, caps, and coils. It joins spheres with cylinders; never use stacked rings or disconnected spheres.\n- Dimensions are mm: radius = ridge centerline radius; pitch = axial travel/turn; tube_radius = half ridge thickness; handedness = 1 (right) or -1 (left).\n- For a hand control use thread_hand = 1; // [-1:Left, 1:Right] and pass it directly. Ternary expressions (?:) are unsupported.\n- Embed the ridge in the shaft/neck (e.g. radius = shaft_radius). Hollow fittings: subtract the bore from body + thread. Internal threads: subtract the helix from the socket wall with mating clearance. Mating parts need matching pitch and handedness.\n- Expose diameter, pitch, thread length, and thickness in Customizer; derive sampling. Keep 2 * tube_radius < pitch and allow tube_radius beyond both path endpoints for rounded ends. This is a rounded profile, not ISO/metric V-thread; never claim standards compliance.\n- Build heads/grips separately using cylinders, tapered cylinders, and radial ribs/cutters, not thread samples.\n\nmodule helical_ridge(radius, pitch, turns, tube_radius, handedness) {\n  if (radius > 0 && pitch > 0 && turns > 0 && tube_radius > 0) {\n    // Limit chord error to 0.03 mm or 3% of the ridge radius.\n    chord_error = min(0.03, tube_radius * 0.03);\n    steps_per_turn = max(48, ceil(180 / acos(1 - min(1, chord_error / radius))));\n    steps = ceil(turns * steps_per_turn);\n    for (i = [0:steps]) {\n      angle = handedness * 360 * turns * i / steps;\n      p = [radius * cos(angle), radius * sin(angle), pitch * turns * i / steps];\n      translate(p) sphere(r = tube_radius);\n      if (i < steps) {\n        next_angle = handedness * 360 * turns * (i + 1) / steps;\n        dx = radius * cos(next_angle) - p[0];\n        dy = radius * sin(next_angle) - p[1];\n        dz = pitch * turns / steps;\n        segment_length = sqrt(dx * dx + dy * dy + dz * dz);\n        translate(p)\n          rotate([0, acos(dz / segment_length), atan2(dy, dx)])\n            cylinder(h = segment_length, r = tube_radius);\n      }\n    }\n  }\n}\n\nPARAMETERS\n- Make important top-level constants user-adjustable with OpenSCAD Customizer comments:\n  - Numbers with ranges: body_radius = 12; // [4:0.5:40]\n  - Counts: leg_count = 4; // [2:8]\n  - Booleans: show_base = true;\n  - Options: style = \"round\"; // [round, sharp, chunky]\n- Prefer stable size/count/ratio/style controls; keep fragile dependent dimensions as local expressions.\n\nSUPPORTED SUBSET\n- Export is best-effort from a supported CSG subset: cube, sphere, cylinder, translate, rotate, color, union, difference, intersection, and goop.\n- difference() cutters become subtractive primitives, so use clean cutters that fully pass through the target solid.\n- Unsupported OpenSCAD may validate but can warn or skip during export; prefer the supported subset above.";
+export declare const OPENSCAD_INSTRUCTIONS = "Use womp.openscad to create or update one adjustable CAD part. Read an existing part before editing it.\nKeep follow-up edits to an existing CAD part in that part unless the user asks to change representation.\nread(0) returns the root container for creation. read(id) returns an existing\npart or container. Use the scene-editing skill when available to discover other scene objects.\ncreate requires a parentId (0 means the scene root); update requires the existing objectId.\nBoth methods compile the effective source and parameters, apply one undoable change, and check wall thickness.\nPreserve intervening user changes. Ask the user when the intended edits conflict or remain ambiguous.\nExisting overrides are preserved for parameters still present;\nparams supplies overrides to merge into the current values. Keep top-level parameter names stable.\nReturn complete source on every update and preserve unrelated geometry. Never create a replacement to repair a part.\nCompile errors return status rejected without applying. Creation requires validated placement bounds.\nAn update may apply with validation unavailable; that is a diagnostic, not proof that it compiles.\nAn applied part remains in the scene even if thickness fails or is unavailable. Use the returned objectId for repairs.\nWhen thickness fails, increase thin walls and fragile connections while preserving requested dimensions and openings.\nAt most four applied printability repairs are allowed per logical build/edit in this run. Respect the returned limit;\ndo not bypass it by recreating the part. Unavailable thickness is not a pass. Report unresolved issues honestly.\nBuild connected geometry and avoid features below 1.4 mm. OpenSCAD source dimensions use millimetres,\nindependently of the scene display unit. New parts are centered and grounded from compiled bounds.\nOpenSCAD is Z-up; the scene-editing API is Y-up. For an unrotated part at its creation-time scale and an identity parent transform,\na source point [x,y,z] mm maps to scene {x:y, y:z, z:-x} times the scene.unitMultiplier inside the access callback, plus nodeGetPos(partId).\nAccount for the part and parent transforms when they are not identity; source coordinates are not scene coordinates.\nAfter a major change, use the scene screenshot API and emitImage to inspect the applied result.\nA committed effect can survive later failure. Inspect receipts and the existing part before retrying.\nDo not print source in the final answer unless requested; describe the result and any unresolved diagnostics.\n\nRUNTIME\n- Produce a single OpenSCAD file.\n- Do not use imports, includes, external libraries, text, hull, minkowski, polyhedron, or extrudes.\n- Prefer cube, sphere, cylinder, translate, rotate, color, union, difference, and intersection.\n- Do not use $fn, $fa, or $fs; tessellation controls are ignored.\n- Build faceted geometry by combining supported primitives with CSG operations and transforms.\n- Do not use scale(); size primitives directly instead.\n- Use color(\"#rrggbb\") around distinct visible parts.\n- Keep the model centered near the origin and use millimeter-like units consistently.\n- For organic blending, use goop(radius_mm) { ... }. The radius is in millimetres before parent scaling, matching scene-editing; do not multiply by project unitMultiplier. It is ordered: the first child seeds with blend 0, and each later child uses radius_mm.\n- Put core primitives first. Avoid wrapping goop children in union() when they should blend individually.\n\nSCREWS AND THREADS\n- Copy helical_ridge below for rounded screw threads, necks, caps, and coils. It joins spheres with cylinders; never use stacked rings or disconnected spheres.\n- Dimensions are mm: radius = ridge centerline radius; pitch = axial travel/turn; tube_radius = half ridge thickness; handedness = 1 (right) or -1 (left).\n- For a hand control use thread_hand = 1; // [-1:Left, 1:Right] and pass it directly. Ternary expressions (?:) are unsupported.\n- Embed the ridge in the shaft/neck (e.g. radius = shaft_radius). Hollow fittings: subtract the bore from body + thread. Internal threads: subtract the helix from the socket wall with mating clearance. Mating parts need matching pitch and handedness.\n- Expose diameter, pitch, thread length, and thickness in Customizer; derive sampling. Keep 2 * tube_radius < pitch and allow tube_radius beyond both path endpoints for rounded ends. This is a rounded profile, not ISO/metric V-thread; never claim standards compliance.\n- Build heads/grips separately using cylinders, tapered cylinders, and radial ribs/cutters, not thread samples.\n\nmodule helical_ridge(radius, pitch, turns, tube_radius, handedness) {\n  if (radius > 0 && pitch > 0 && turns > 0 && tube_radius > 0) {\n    // Limit chord error to 0.03 mm or 3% of the ridge radius.\n    chord_error = min(0.03, tube_radius * 0.03);\n    steps_per_turn = max(48, ceil(180 / acos(1 - min(1, chord_error / radius))));\n    steps = ceil(turns * steps_per_turn);\n    for (i = [0:steps]) {\n      angle = handedness * 360 * turns * i / steps;\n      p = [radius * cos(angle), radius * sin(angle), pitch * turns * i / steps];\n      translate(p) sphere(r = tube_radius);\n      if (i < steps) {\n        next_angle = handedness * 360 * turns * (i + 1) / steps;\n        dx = radius * cos(next_angle) - p[0];\n        dy = radius * sin(next_angle) - p[1];\n        dz = pitch * turns / steps;\n        segment_length = sqrt(dx * dx + dy * dy + dz * dz);\n        translate(p)\n          rotate([0, acos(dz / segment_length), atan2(dy, dx)])\n            cylinder(h = segment_length, r = tube_radius);\n      }\n    }\n  }\n}\n\nPARAMETERS\n- Make important top-level constants user-adjustable with OpenSCAD Customizer comments:\n  - Numbers with ranges: body_radius = 12; // [4:0.5:40]\n  - Counts: leg_count = 4; // [2:8]\n  - Booleans: show_base = true;\n  - Options: style = \"round\"; // [round, sharp, chunky]\n- Prefer stable size/count/ratio/style controls; keep fragile dependent dimensions as local expressions.\n\nSUPPORTED SUBSET\n- Export is best-effort from a supported CSG subset: cube, sphere, cylinder, translate, rotate, color, union, difference, intersection, and goop.\n- difference() cutters become subtractive primitives, so use clean cutters that fully pass through the target solid.\n- Unsupported OpenSCAD may validate but can warn or skip during export; prefer the supported subset above.";

package/dist/capabilities/openscad-instructions.js

@@ -1,82 +1,81 @@
-export const OPENSCAD_INSTRUCTIONS = `Use womp.openscad to create or update one adjustable CAD part. Read an existing part before editing it.
-Keep follow-up edits to an existing CAD part in that part unless the user asks to change representation.
-read(0) returns the root container and a scene revision for creation. read(id) returns an existing
-part or container. Use the scene-editing skill when available to discover other scene objects.
-create requires a parentId (0 means the scene root); update requires the existing objectId.
-Both methods compile the effective source and parameters, apply one undoable change, and check wall thickness.
-A stale revision requires reading again and adapting the edit while preserving intervening user changes.
-Ask the user only when the intended edits conflict or remain ambiguous; never reuse the old replacement
-with a refreshed revision token. Existing overrides are preserved for parameters still present;
-params supplies overrides to merge into the current values. Keep top-level parameter names stable.
-Return complete source on every update and preserve unrelated geometry. Never create a replacement to repair a part.
-Compile errors return status rejected without applying. Creation requires validated placement bounds.
-An update may apply with validation unavailable; that is a diagnostic, not proof that it compiles.
-An applied part remains in the scene even if thickness fails or is unavailable. Use the returned objectId for repairs.
-When thickness fails, increase thin walls and fragile connections while preserving requested dimensions and openings.
-At most four applied printability repairs are allowed per logical build/edit in this run. Respect the returned limit;
-do not bypass it by recreating the part. Unavailable thickness is not a pass. Report unresolved issues honestly.
-Build connected geometry and avoid features below 1.4 mm. OpenSCAD source dimensions use millimetres,
-independently of the scene display unit. New parts are centered and grounded from compiled bounds.
-OpenSCAD is Z-up; the scene-editing API is Y-up. For an unrotated part at its creation-time scale and an identity parent transform,
-a source point [x,y,z] mm maps to scene {x:y, y:z, z:-x} times the unitMultiplier returned by scene.read(), plus nodeGetPos(partId).
-Account for the part and parent transforms when they are not identity; source coordinates are not scene coordinates.
-After a major change, use the scene screenshot API and emitImage to inspect the applied result.
-A committed effect can survive later failure. Inspect receipts and the existing part before retrying.
-Do not print source in the final answer unless requested; describe the result and any unresolved diagnostics.
-
-RUNTIME
-- Produce a single OpenSCAD file.
-- Do not use imports, includes, external libraries, text, hull, minkowski, polyhedron, or extrudes.
-- Prefer cube, sphere, cylinder, translate, rotate, color, union, difference, and intersection.
-- Do not use $fn, $fa, or $fs; tessellation controls are ignored.
-- Build faceted geometry by combining supported primitives with CSG operations and transforms.
-- Do not use scale(); size primitives directly instead.
-- Use color("#rrggbb") around distinct visible parts.
-- Keep the model centered near the origin and use millimeter-like units consistently.
-- For organic blending, use goop(radius_mm) { ... }. The radius is in millimetres before parent scaling, matching scene-editing; do not multiply by project unitMultiplier. It is ordered: the first child seeds with blend 0, and each later child uses radius_mm.
-- Put core primitives first. Avoid wrapping goop children in union() when they should blend individually.
-
-SCREWS AND THREADS
-- Copy helical_ridge below for rounded screw threads, necks, caps, and coils. It joins spheres with cylinders; never use stacked rings or disconnected spheres.
-- Dimensions are mm: radius = ridge centerline radius; pitch = axial travel/turn; tube_radius = half ridge thickness; handedness = 1 (right) or -1 (left).
-- For a hand control use thread_hand = 1; // [-1:Left, 1:Right] and pass it directly. Ternary expressions (?:) are unsupported.
-- Embed the ridge in the shaft/neck (e.g. radius = shaft_radius). Hollow fittings: subtract the bore from body + thread. Internal threads: subtract the helix from the socket wall with mating clearance. Mating parts need matching pitch and handedness.
-- Expose diameter, pitch, thread length, and thickness in Customizer; derive sampling. Keep 2 * tube_radius < pitch and allow tube_radius beyond both path endpoints for rounded ends. This is a rounded profile, not ISO/metric V-thread; never claim standards compliance.
-- Build heads/grips separately using cylinders, tapered cylinders, and radial ribs/cutters, not thread samples.
-
-module helical_ridge(radius, pitch, turns, tube_radius, handedness) {
-  if (radius > 0 && pitch > 0 && turns > 0 && tube_radius > 0) {
-    // Limit chord error to 0.03 mm or 3% of the ridge radius.
-    chord_error = min(0.03, tube_radius * 0.03);
-    steps_per_turn = max(48, ceil(180 / acos(1 - min(1, chord_error / radius))));
-    steps = ceil(turns * steps_per_turn);
-    for (i = [0:steps]) {
-      angle = handedness * 360 * turns * i / steps;
-      p = [radius * cos(angle), radius * sin(angle), pitch * turns * i / steps];
-      translate(p) sphere(r = tube_radius);
-      if (i < steps) {
-        next_angle = handedness * 360 * turns * (i + 1) / steps;
-        dx = radius * cos(next_angle) - p[0];
-        dy = radius * sin(next_angle) - p[1];
-        dz = pitch * turns / steps;
-        segment_length = sqrt(dx * dx + dy * dy + dz * dz);
-        translate(p)
-          rotate([0, acos(dz / segment_length), atan2(dy, dx)])
-            cylinder(h = segment_length, r = tube_radius);
-      }
-    }
-  }
-}
-
-PARAMETERS
-- Make important top-level constants user-adjustable with OpenSCAD Customizer comments:
-  - Numbers with ranges: body_radius = 12; // [4:0.5:40]
-  - Counts: leg_count = 4; // [2:8]
-  - Booleans: show_base = true;
-  - Options: style = "round"; // [round, sharp, chunky]
-- Prefer stable size/count/ratio/style controls; keep fragile dependent dimensions as local expressions.
-
-SUPPORTED SUBSET
-- Export is best-effort from a supported CSG subset: cube, sphere, cylinder, translate, rotate, color, union, difference, intersection, and goop.
-- difference() cutters become subtractive primitives, so use clean cutters that fully pass through the target solid.
+export const OPENSCAD_INSTRUCTIONS = `Use womp.openscad to create or update one adjustable CAD part. Read an existing part before editing it.
+Keep follow-up edits to an existing CAD part in that part unless the user asks to change representation.
+read(0) returns the root container for creation. read(id) returns an existing
+part or container. Use the scene-editing skill when available to discover other scene objects.
+create requires a parentId (0 means the scene root); update requires the existing objectId.
+Both methods compile the effective source and parameters, apply one undoable change, and check wall thickness.
+Preserve intervening user changes. Ask the user when the intended edits conflict or remain ambiguous.
+Existing overrides are preserved for parameters still present;
+params supplies overrides to merge into the current values. Keep top-level parameter names stable.
+Return complete source on every update and preserve unrelated geometry. Never create a replacement to repair a part.
+Compile errors return status rejected without applying. Creation requires validated placement bounds.
+An update may apply with validation unavailable; that is a diagnostic, not proof that it compiles.
+An applied part remains in the scene even if thickness fails or is unavailable. Use the returned objectId for repairs.
+When thickness fails, increase thin walls and fragile connections while preserving requested dimensions and openings.
+At most four applied printability repairs are allowed per logical build/edit in this run. Respect the returned limit;
+do not bypass it by recreating the part. Unavailable thickness is not a pass. Report unresolved issues honestly.
+Build connected geometry and avoid features below 1.4 mm. OpenSCAD source dimensions use millimetres,
+independently of the scene display unit. New parts are centered and grounded from compiled bounds.
+OpenSCAD is Z-up; the scene-editing API is Y-up. For an unrotated part at its creation-time scale and an identity parent transform,
+a source point [x,y,z] mm maps to scene {x:y, y:z, z:-x} times the scene.unitMultiplier inside the access callback, plus nodeGetPos(partId).
+Account for the part and parent transforms when they are not identity; source coordinates are not scene coordinates.
+After a major change, use the scene screenshot API and emitImage to inspect the applied result.
+A committed effect can survive later failure. Inspect receipts and the existing part before retrying.
+Do not print source in the final answer unless requested; describe the result and any unresolved diagnostics.
+
+RUNTIME
+- Produce a single OpenSCAD file.
+- Do not use imports, includes, external libraries, text, hull, minkowski, polyhedron, or extrudes.
+- Prefer cube, sphere, cylinder, translate, rotate, color, union, difference, and intersection.
+- Do not use $fn, $fa, or $fs; tessellation controls are ignored.
+- Build faceted geometry by combining supported primitives with CSG operations and transforms.
+- Do not use scale(); size primitives directly instead.
+- Use color("#rrggbb") around distinct visible parts.
+- Keep the model centered near the origin and use millimeter-like units consistently.
+- For organic blending, use goop(radius_mm) { ... }. The radius is in millimetres before parent scaling, matching scene-editing; do not multiply by project unitMultiplier. It is ordered: the first child seeds with blend 0, and each later child uses radius_mm.
+- Put core primitives first. Avoid wrapping goop children in union() when they should blend individually.
+
+SCREWS AND THREADS
+- Copy helical_ridge below for rounded screw threads, necks, caps, and coils. It joins spheres with cylinders; never use stacked rings or disconnected spheres.
+- Dimensions are mm: radius = ridge centerline radius; pitch = axial travel/turn; tube_radius = half ridge thickness; handedness = 1 (right) or -1 (left).
+- For a hand control use thread_hand = 1; // [-1:Left, 1:Right] and pass it directly. Ternary expressions (?:) are unsupported.
+- Embed the ridge in the shaft/neck (e.g. radius = shaft_radius). Hollow fittings: subtract the bore from body + thread. Internal threads: subtract the helix from the socket wall with mating clearance. Mating parts need matching pitch and handedness.
+- Expose diameter, pitch, thread length, and thickness in Customizer; derive sampling. Keep 2 * tube_radius < pitch and allow tube_radius beyond both path endpoints for rounded ends. This is a rounded profile, not ISO/metric V-thread; never claim standards compliance.
+- Build heads/grips separately using cylinders, tapered cylinders, and radial ribs/cutters, not thread samples.
+
+module helical_ridge(radius, pitch, turns, tube_radius, handedness) {
+  if (radius > 0 && pitch > 0 && turns > 0 && tube_radius > 0) {
+    // Limit chord error to 0.03 mm or 3% of the ridge radius.
+    chord_error = min(0.03, tube_radius * 0.03);
+    steps_per_turn = max(48, ceil(180 / acos(1 - min(1, chord_error / radius))));
+    steps = ceil(turns * steps_per_turn);
+    for (i = [0:steps]) {
+      angle = handedness * 360 * turns * i / steps;
+      p = [radius * cos(angle), radius * sin(angle), pitch * turns * i / steps];
+      translate(p) sphere(r = tube_radius);
+      if (i < steps) {
+        next_angle = handedness * 360 * turns * (i + 1) / steps;
+        dx = radius * cos(next_angle) - p[0];
+        dy = radius * sin(next_angle) - p[1];
+        dz = pitch * turns / steps;
+        segment_length = sqrt(dx * dx + dy * dy + dz * dz);
+        translate(p)
+          rotate([0, acos(dz / segment_length), atan2(dy, dx)])
+            cylinder(h = segment_length, r = tube_radius);
+      }
+    }
+  }
+}
+
+PARAMETERS
+- Make important top-level constants user-adjustable with OpenSCAD Customizer comments:
+  - Numbers with ranges: body_radius = 12; // [4:0.5:40]
+  - Counts: leg_count = 4; // [2:8]
+  - Booleans: show_base = true;
+  - Options: style = "round"; // [round, sharp, chunky]
+- Prefer stable size/count/ratio/style controls; keep fragile dependent dimensions as local expressions.
+
+SUPPORTED SUBSET
+- Export is best-effort from a supported CSG subset: cube, sphere, cylinder, translate, rotate, color, union, difference, intersection, and goop.
+- difference() cutters become subtractive primitives, so use clean cutters that fully pass through the target solid.
 - Unsupported OpenSCAD may validate but can warn or skip during export; prefer the supported subset above.`;

package/dist/capabilities/openscad.d.ts

@@ -6,13 +6,11 @@ export type OpenScadTarget = {
     title: string;
     source: string | null;
     params: Record<string, unknown>;
-    revision: string;
 };
 export type OpenScadSource = {
     source: string;
     title: string;
     params?: Record<string, unknown>;
-    expectedRevision: string;
 };
 export type OpenScadCreate = OpenScadSource & {
     parentId: number;
@@ -35,7 +33,6 @@ export type OpenScadPrintability = {
 export type OpenScadWriteResult = {
     status: "applied" | "rejected";
     objectId?: number;
-    revision: string;
     validation: OpenScadValidation;
     printability?: OpenScadPrintability;
     repairAttempts: number;

package/dist/capabilities/openscad.js

@@ -6,7 +6,6 @@ const source = z.object({
     source: z.string().trim().min(1),
     title: z.string().trim().min(1),
     params: z.record(z.string(), z.json()).optional(),
-    expectedRevision: z.string().min(1),
 });
 const create = source.extend({ parentId: nodeId }).strict();
 const update = source.extend({ objectId: nodeId.positive() }).strict();
@@ -21,7 +20,7 @@ export function createOpenScadCapability(host) {
                 declarations: WOMP_OPENSCAD_DECLARATIONS,
                 examples: [
                     "const part = await womp.openscad.read(objectId); print(part);",
-                    "const part = await womp.openscad.read(objectId); print(await womp.openscad.update({ objectId: part.objectId, source: part.source, title: part.title, params: { width: 30 }, expectedRevision: part.revision }));",
+                    "const part = await womp.openscad.read(objectId); print(await womp.openscad.update({ objectId: part.objectId, source: part.source, title: part.title, params: { width: 30 } }));",
                 ],
             },
         ],

package/dist/capabilities/scene.d.ts

@@ -1,10 +1,7 @@
-import type { CameraConfiguration } from "@womp/kakapo-sdk";
 import type { ArtifactRef, SparkCapability, SparkInvocationContext } from "../capabilities.js";
 import type { WompSceneDraft } from "./scene-draft.js";
 import { type SceneHealth } from "../scene-health.js";
 export type SceneSnapshot = {
-    health: SceneHealth;
-    revision: string;
     projectId?: number;
     unitMultiplier: number;
     nodes: {
@@ -14,23 +11,19 @@ export type SceneSnapshot = {
         parentId: number;
     }[];
 };
-export type SceneScreenshotOptions = {
-    /** Position/target and orthographic width use Womp scene coordinates and current project units. */
-    camera?: CameraConfiguration;
-};
+/** Current viewport only; no framing or camera overrides. */
+export type SceneScreenshotOptions = Record<string, never>;
 export type WompSceneApi = {
-    read(): Promise<SceneSnapshot>;
     health(objectIds?: number[]): Promise<SceneHealth>;
-    edit<T>(callback: (draft: WompSceneDraft) => T | Promise<T>, expectedRevision: string): Promise<T>;
+    access<T>(callback: (draft: WompSceneDraft) => T | Promise<T>): Promise<T>;
     screenshot(options?: SceneScreenshotOptions): Promise<ArtifactRef>;
 };
 export interface SceneHealthHost {
     read(objectIds: number[], context: SparkInvocationContext): Promise<SceneHealth>;
 }
 export interface SceneHost {
-    read(context: SparkInvocationContext): Promise<Omit<SceneSnapshot, "health">>;
     health?: SceneHealthHost["read"];
-    edit<T>(callback: (draft: WompSceneDraft) => T | Promise<T>, expectedRevision: string, context: SparkInvocationContext): Promise<T>;
+    access<T>(callback: (draft: WompSceneDraft) => T | Promise<T>, context: SparkInvocationContext): Promise<T>;
     screenshot(options: SceneScreenshotOptions, context: SparkInvocationContext): Promise<ArtifactRef>;
 }
 export declare function createSceneCapability(host: SceneHost): SparkCapability<WompSceneApi>;