Repository navigation
Docs catch up with 0.8.0 and the explore work - #88
Merged
Merged
Conversation
bug-garden/tools/check-bench.mjs ignored the `absent_text` the harness enforces, so bench-delete-plant passed without checking Aloe Vera was gone, and it matched case-sensitively where the harness does not. It now does both as the harness does. The skill's exit-4 line said a skip meant "no build, no matching device". Both of those are refused at enqueue (exit 2 or 5) and never run; exit 4 is a job that was claimed and reported `skipped:`. Regenerated SKILL.md. seed-schedules: the comment counted three explore schedules (there are four) and still expected the TV one to wait for a Fire TV test; Fire TV is parked. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The 0.8.0 changelog left out everything merged before the cut that was not the control API: exploratory QA (#70, #73, #74, #77, #78), the device art (#69) and the two runner-android fixes (#75, #76). They get an "Also in this release" section with their not-verified items. A read of every page against the code found ~60 stale spots. The main ones: - Explore: card counts, Holo4's measured status, tool_choice and json_schema requirements, the real gateway-key lookup order and env vars, Fire TV parked for tvOS, the night-queue jobs and priorities, budgets, auth on POST /findings. - Front door: Explore in the README, index and getting-started; 34 workloads, not 28; "no authentication, by design" now says by default and links the opt-in tokens; the Roku driver; strict validation is /api/v1 only; CI-verified items in platforms; ultra and the S8+ in the lab runbook; #45's caveat removed (fixed in 0.6.1). - API and integration: undocumented routes and /metrics, the guards under FLEET_AUTH, CLI flags and env vars, the /api/v1-only owner and shell checks stated plainly. Guides lead with fleetr instead of the deprecated routes, say what ci-enqueue gets wrong (a skip exits 0; no token), add the required app.build to examples, and split an array "enqueue" no endpoint accepts into three specs. - A tvloop section in workloads/host.md, which the catalog already linked. - mkdocs nav listed the ultra runbook twice; it stays under Exploratory QA and the Deploy overview links it. Not done here: the three mermaid diagrams in the explore pages, the 28- workload figure and the pre-device-art dashboard screenshots. They need figures rendered, not text edits. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Owner
Author
|
Self-review: strict docs build and all 19 checks pass; code changes are a generated skill string, a comment and a standalone checker (tested against a dump). Merging per the plan (LOOP-PLAN Phase 0). |
addisdev
added a commit
that referenced
this pull request
Oct 16, 2026
This was referenced Oct 16, 2026
Merged
3 of 5 tasks
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.
Summary
fleetrinstead of the deprecated routes.tvloopsection, which the catalog already linked to.check-bench.mjsreadsabsent_textand matches case-insensitively, as the harness does.Gaps these docs now admit, to be fixed in code later
POST /api/jobs/:id/cancel|retryhave no owner check.shellfrom aruntoken.ci-enqueueexits 0 on a skipped job and sends no token.fleet/package.jsonbinpoints at a file the build does not produce.version.mjs.Not in this PR
All of these need figures rendered, not text.
Test plan
mkdocs build --strict(Material 9.7.7) is clean.fleetskill test passes (3/3) afternpm run skill:gen.check-bench.mjsfails a dump that still shows "Aloe Vera" and passes one that doesn't.fleetandcollectorsuites (nocollector/node_modulesin this worktree). The code changes are a comment, one generated string, and a standalone script.🤖 Generated with Claude Code