Skip to content

DRAFT: explicit v1 Artist Context workflow node (first part of #252) - #6

Draft
JayceeB1 wants to merge 11 commits into
integration/agent-asset-context-2026-10-10from
integration/artist-intent-core-2026-10-10
Draft

JayceeB1 wants to merge 11 commits into
integration/agent-asset-context-2026-10-10from
integration/artist-intent-core-2026-10-10

Conversation

@JayceeB1

@JayceeB1 JayceeB1 commented Oct 9, 2026

Copy link
Copy Markdown
Owner

Goal

First bounded, explicitly opt-in slice of upstream #252, originally contributed by @OmarB97. This stacks on our PR #5 and does not change fork main or the upstream.

The original lightningpixel#252 mixes immutable artist-intent records, automatic multi-stage propagation, live texture reload, viewer UI changes and proof assets. Here we isolate the modly.intent.v1 recording and carry contract to test safely before changing core workflows.

Included

  • Built-in intent-travel process extension with Remember Artist Context (image → staged image + sidecar) and Carry Artist Context (asset with existing intent + distinct existing workspace destination asset).
  • Stable intent_id, artist words, project/style/movement facts, stage events, source-image basename, conflict-checking and append-only-style history merge. Each start captures a new request instead of overwriting prior records.
  • Prevent writing or reading the destination outside the real workspace. Refuse symlinked metadata sidecars and staging directories that escape the workspace; cap image size at 64 MiB and sidecars at 1 MiB.
  • Add 9 Node regression test cases to npm run test:node for staging, intent identity/carry, outside or missing destinations, source type, conflicting intent, size caps, and link escapes.
  • Builtins TypeScript compiler includes the new extension.

Excluded by design

  • No automatic interception of all image→model workflows. Generated meshes do not yet automatically inherit an image's intent record. The carry action only works on a source asset that already contains the record.
  • No live texture hot reload, watcher, UI replacement, or proof images from Carry artist intent through generated assets lightningpixel/modly#252.
  • No claim that a multi-stage production automatically retains creative intent yet.

Verification

  • Source is isolated in a review branch with a documented scope and ledger.
  • Regression tests are written and wired into the Node suite.
  • Targeted tests with installed repository dependencies and overall lint/typecheck/build.
  • End-to-end Electron/Windows workflow test including a real source image, generated GLB, and conflicts.
  • Security review, including filesystem races and externally modified user data.

Detailed scope and limitations: docs/experimental-artist-intent.md.
Other integrations: docs/experimental-library-integration-ledger.md.

Draft only. Do not merge to a releasable branch or run against valuable storage until the combined validation gates pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant