Skip to content

Fix guidance and navigation across all ten demo walkthroughs - #446

Merged
Sev7eNup merged 2 commits into
mainfrom
fix/tutorial-guidance
Sep 28, 2026
Merged

Sev7eNup merged 2 commits into
mainfrom
fix/tutorial-guidance

Conversation

@Sev7eNup

@Sev7eNup Sev7eNup commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

What & why

Make all ten guided demo tasks easier to follow and reliably finish. Instructions now match the visible controls, file delivery has its own completion with optional diagnosis, and finished tasks offer the next task and a working overview link.

How

  • Keep the live cancellation exercise running until the learner cancels it; preserve tutorial buttons during unrelated updates.
  • Open the parallel run directly in a larger Gantt view, highlight relevant steps without reducing text contrast, and clarify simulated workflow notes in German and English.
  • Correct task progress, unknown machine status, zero workflow counts and maintenance windows that cross midnight.
  • Follow execution deep-link changes and read published workflow parameters from the saved definition while the canvas loads.

Testing

  • 355 scoped demo, editor, execution and maintenance unit tests pass, including regression coverage for early Run clicks and overview navigation.
  • All 43 demo browser tests pass, covering tutorials in both languages, narrow viewports, smoke tests and routing.
  • All 21 documentation count checks pass, including the updated frontend test count.
  • Manual review of all ten public tutorials, with local follow-up checks of changed flows.
  • Website build and upload verified against the served asset hashes; Tour 02 contrast and the overview link checked on the live website.
  • Scoped ESLint passes; existing TanStack Virtual compiler warnings remain.

Both languages are updated. These changes are already deployed to the public website; this PR brings the reviewed source into main.

@Sev7eNup
Sev7eNup enabled auto-merge (squash) September 28, 2026 18:29
}

const unsubscribe = subscribeWorld(update);
const resize = new ResizeObserver(() => document.body.style.setProperty('--np-tour-height', `${panel.getBoundingClientRect().height}px`));
@Sev7eNup
Sev7eNup merged commit 504954b into main Sep 28, 2026
17 checks passed
@Sev7eNup
Sev7eNup deleted the fix/tutorial-guidance branch September 28, 2026 18:49
Sev7eNup added a commit that referenced this pull request Oct 3, 2026
#452)

Daily documentation sweep over the pull requests active since
2026-09-28T01:03:39Z. Three surfaces had drifted.

#448 allowed same-origin downloads in the Electron shell, added the
computer-use acceptance gate and made globals resolve inside a custom
activity definition:

- deploy/desktop/README.md and content/{de,en}/deployment/desktop.md still
  listed downloads among the things the shell blocks outright.
- The tray entry the shell README tells you to use is now "Quit NodePilot".
- CLAUDE.md counted three mandatory release runs; scripts/release-lab/
  computer-use/ is a fourth, and RELEASING.md step 6 describes it.
- docs/custom-activities.md and content/{de,en}/concepts/data-bus.md
  described the globals contract for runScript only, although a
  {{globals.X}} in a ScriptTemplate or an input default now resolves and
  fails the step when it is missing or undecryptable.

#446 renamed the designer's run button from "Test run" to "Run"
("Ausführen"). docs/workflow-designer-features.md and both language
versions of designer/properties-modes.md still named the old label,
including in their keyboard-shortcut tables.

Verified against the checked-out main rather than the pull request diffs.
The counts these changes could have invalidated all still match the code:
27 activity types, 10 skins, 242 vitest files, 77 E2E specs. #447 carried
its own documentation across docs/claude-reference.md and both copies of
activities-reference.md, so the startProgram drain semantics needed no
edit.


Claude-Session: https://claude.ai/code/session_018Hdn3USoyZs9C2DQ3LboAA

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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