Skip to content

Salvage from the Mac parking branch: existing-app-change skill, cloud-prompt correction, workflow §21, JUMP TO STOP row, BUG-140 - #20

Merged
MendixMau merged 3 commits into
masterfrom
queue/mac-divergence-salvage
Sep 8, 2026
Merged

Salvage from the Mac parking branch: existing-app-change skill, cloud-prompt correction, workflow §21, JUMP TO STOP row, BUG-140#20
MendixMau merged 3 commits into
masterfrom
queue/mac-divergence-salvage

Conversation

@MendixMau

Copy link
Copy Markdown
Owner

Direct lane

What changed and why: the pieces of wip/local-divergence-20260901 (draft PR #2) that master genuinely lacks, ported onto current master. 8 of that branch's 11 commits were already on master; this carries the rest:

  • 372e2ac cherry-picked (author kept): the cloud/mobile prompt commands no longer claim the build gate needs Studio Pro — consistent with the cloud lane and "Where you run this".
  • skills/existing-app-change.md — new complete skill: changing a live app that has no BRDs, architecture doc or wireframes (Path D knowledge base from the live model, stages 2–4 over the changed slice plus blast radius, Track B regression baseline as precondition). Routing row (ba, architect · P,0 · ondemand), pointer from existing-app-assurance.md, deliverables given denominators.
  • learned-workflow-patterns.md §21 — the v0.20.0 re-probe, reconciled rather than pasted: the branch said DECISION was cleared; master's BUG-76 retest reproduced the corruption byte-for-byte the same day, so §21 records DECISION as disputed / shape-dependent, the boundary-timer clearance scoped to the non-interrupting form, and CE6681 attributed to both known causes.
  • learned-mdl-preflight.md STOP row 24: JUMP TO target validation.
  • Bug log: BUG-140 (full regeneration never wires a parameterised microflow datasource on a top-level dataview; number provisional, assigned at merge), addenda under BUG-109/110/118 for the earlier observations, and a "Cleared on v0.20.0" section with BUG-76 marked disputed.
  • All three stale project-profile.mdPROJECT.md references fixed; CLAUDE.md two-test-suites note.

Not ported, on purpose: docs/progress/* (one machine's session state), the .gitignore change (would untrack files master tracks), and everything master already had.

Merge-queue review: ported by a worker in an isolated worktree; five gates green; one client-ish reference genericised. Open item for a human with a v0.20.0 binary: bisect the DECISION shape — the BUG-76 contradiction is recorded in three places, not resolved. After merge: close draft PR #2 and delete wip/local-divergence-20260901.

  • No client data in the diff
  • No instrument changed
  • New skill: routing row added, re-rendered
  • CHANGELOG lines present, credited

🤖 Generated with Claude Code

https://claude.ai/code/session_01VJgWP5vEoAsNsJYqCDMGNw


Generated by Claude Code

MendixMau and others added 3 commits September 8, 2026 04:12
commands/mobile-dev-loop-prompt.md asserted that the build gate requires
Studio Pro and that a headless container therefore cannot run it. That is
wrong, and it was stopping cloud sessions at a boundary that does not exist:
the gate is mxbuild, a plain binary that project-bin/exec.sh invokes directly,
and exec.sh's only Studio Pro coupling is a lock check (refuse to write while
SP holds the .mpr) which is a no-op where no SP exists.

Both prompts also predate mxcli run --local / --hub / --test-endpoint, which
the toolkit did not mention anywhere. A container can now stand the app up
itself and publish it at a public URL, so the hand-rolled ngrok/cloudflared
tunnel is no longer the only route in.

- mobile-dev-loop-prompt.md: dated correction note; DO-NOT list reframed as
  out-of-scope-for-this-prompt rather than impossible-here.
- mobile-auto-test-prompt.md: stand-it-up-here is now the preferred option.
- testing-shape.md: mxcli run --local flag table, runtime.log note.
- e2e-harness-base.md: BASE_URL no longer assumes localhost:8080.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
…rkflow re-probe, BUG-140

Hand-ported from wip/local-divergence-20260901 (7cd08ca) onto current master — only what a
prior review found worth keeping, reconciled against what master has learned since:

- skills/existing-app-change.md, new: the fourth entry mode (change a live app with no BRDs);
  routed ba,architect / P,0 / ondemand / spine, surfaces re-rendered; existing-app-assurance.md
  points here for "change the app you just audited".
- learned-workflow-patterns.md §21 (v0.20.0 re-probe) plus the header pointer, the §8 version-
  table row, §10 step 5 and Notes on scope. DECISION recorded as DISPUTED, not cleared: the
  toolkit's own same-day retest reproduced BUG-76 byte for byte with `decision '1 = 1'`.
  CE6681 now attributed to both causes (forward jump per workflow-structure-rules.md, and the
  self-referencing dangling jump).
- learned-mdl-preflight.md STOP #24 (JUMP TO target validation); WITH-clause gotcha stamped
  v0.16.0-only.
- bug-logs: BUG-140 (top-level dataview parameterized microflow datasource never wired on full
  regen) — number assigned at merge, clear of the 127+ block in flight elsewhere. The branch's
  BUG-97/98/99 were master's BUG-118/109/110: addenda under those, not new entries. "Cleared on
  v0.20.0" section added with BUG-76 marked disputed; a matching note under BUG-76.
- checkpoints: all three stale project-profile.md references → PROJECT.md (the architecture one
  was not on the branch; found by grep).
- CLAUDE.md: the two-suites testing note, checked still accurate against tests/.

Not ported: docs/progress/*, the .gitignore change, and everything master already carried
(bin/sync-project.sh, project-bin/verify-module.sh, project-tests/e2e/design-audit.js,
workflow-patterns §14–20).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VJgWP5vEoAsNsJYqCDMGNw
…e-salvage

# Conflicts:
#	CHANGELOG.md
#	bug-logs/mxcli-bugs.md
@MendixMau
MendixMau merged commit fd60fb8 into master Sep 8, 2026
1 check passed
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.

2 participants