Repository navigation
Conversation
#466 added Teil 86 (the browser demo's ten guided tasks) to docs/testing/E2ETests.md, which now holds 86 Teile. The coverage map in src/nodepilot-ui/e2e/README.md still introduced the catalogue as 85 Teile. DocumentationCountsTests pins the spec-file and doc-page counts but not this one, so nothing caught it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DVZHQ2MF4vhtDyvX5okC3A
This was referenced Oct 5, 2026
Closed
Owner
Author
|
Superseded by #494, which carries the same README fix plus a DocumentationCountsTests row for the part count. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Daily documentation-drift sweep over the PRs active between 2026-10-03T01:03Z and
2026-10-04T03:03Z. One clear, objective drift was found and is fixed here.
src/nodepilot-ui/e2e/README.md#466 appended Teil 86 ("Geführte Aufgaben der Browser-Demo") to
docs/testing/E2ETests.md, which now carries 86 contiguous Teile (1–86). The coveragemap's intro sentence still named 85.
DocumentationCountsTestspins the MCP tool,activity, skin, vitest, e2e-spec and doc-site-page counts, but not this one, so CI had
nothing to catch it with.
No row was added to the coverage table: Teil 86 covers
e2e-demo/specs, which theREADME deliberately keeps out of the table ("The browser demo has its own suite, in
e2e-demo/rather than here") — the same way Teil 85 is handled.How
A one-word count correction. Nothing else in the window required a doc edit:
summaryfromGET /api/triggers/schedule/next-fires,np cron nextand the MCPvalidate_crontool is documented nowhere, so no surface wentstale; the PR updated the maintenance-window line in
docs/claude-reference.mditself. Thedocs-site gap for the new DST and cron-parsing semantics is already tracked in PR-Review #467: Quartz 4 changed when schedules fire, and the documentation website does not say so #473 and is
deliberately not duplicated here.
package.jsonandpackage-contract.test.ts.Two findings were deliberately not auto-applied and are filed as issues instead.
Testing
Documentation only — no code, no test behaviour changed. The guarded counts were
re-derived by hand against the guard formulas and all match
(
e2especs 77, vitest files 242, docs-site EN pages 45).DocumentationCountsTestsdoes not cover the
N Teileclaim; the figures it does cover were verified unchangedDocumentationCountsTestsruns on every PR — Markdown is abackend input, so the backend job is never skipped)
Checklist
main; no direct commits tomain🤖 Generated with Claude Code
https://claude.ai/code/session_01DVZHQ2MF4vhtDyvX5okC3A
Generated by Claude Code