Repository navigation
π docs: resync the Hive docs mirror from hive@v5 - #497
Conversation
Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
β Deploy Preview for hivecommons-docs failed. Why did it fail? β
|
|
hive/awaiting-ci: CI pending β sweep will merge when green. π Hive Agent: β hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88 |
There was a problem hiding this comment.
Looks correct to me β this is a faithful regeneration, not a hand edit.
What I verified at f1ebb39:
- Checked out the PR head, ran
npm ci && npm run sync-hive-docs && npm run sync-sibling-docsfrompackage.json:16-17.git statusis clean afterwards β the committed mirror is byte-identical to what the sync scripts emit today, so nothing was edited by hand or left out. - The body's stated goal matches the diff: e.g.
docs/content/hive/adr/0009-trajectory-review.mdmoves its source links fromsrc/pkg/trajectory/β¦tosrc/crux/trajectory/β¦;https://github.com/hivecommons/hive/blob/v5/src/crux/trajectory/trajectory.goreturns 200. Check internal doc links,markdownlint,Lint and buildand unit tests are green.
One open question, not a defect in the diff: the netlify/hivecommons-docs/deploy-preview status is Deploy Preview failed. (and the three Netlify "rules" statuses with it). The repo's own Lint and build passes on the same head, so I could not tell from here whether this is the content or the Netlify side; the deploy log at the status URL would settle it before merge.
Issue-link note: the body uses Fixes #496 (needs-human: β¦) β the scheduled refresh workflow it mentions still needs a maintainer to add it under .github/workflows/, so #496 will auto-close on merge even though that part is outstanding.
β hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88
There was a problem hiding this comment.
Decision needed: @clubanderson should this PR close #496 on merge (body says Fixes #496), given #496's primary ask β the scheduled hive-docs-refresh workflow β is explicitly left for a maintainer and would lose its open tracker?
The resync itself checks out; the only thing I'd change is the issue link.
Verified
- Every
https://github.com/hivecommons/hive/blob/...link added by this diff (34 unique URLs, including thesrc/crux/...paths #496 lists foradr/0009,adr/0012andrunning-at-level-6.md) returns HTTP 200 as of this review. - The failing Netlify checks are not attributable to this diff:
netlify.tomlruns the samenpm run build(next build) that the passing "Lint and build" check ran at f1ebb39, and the{{ β¦ }}template-looking text indocs/content/promptargs/readme.mdis already onmain(34 occurrences) and deploys there. The Netlify log is external, so I can't say more than that.
Issue link β Fixes vs Refs
#496 is titled "β¦add a scheduled resync workflow" and frames the drift as a gate gap: a hand resync is the symptom fix, the workflow is the ask. This PR is the hand resync, and its own body says the workflow "still has to be added by a maintainer β the hive App cannot push .github/workflows/ files". With Fixes #496, GitHub closes #496 on merge and nothing open records that the workflow was never added β the exact drift #496 describes recurs next time hive moves a file. Refs #496 (needs-human: scheduled hive-docs-refresh workflow still to be added β App cannot push .github/workflows/) keeps the tracker alive and says why. A maintainer's call; I have not changed anything.
β hive: agent=reviewer-2 backend=copilot model=claude-fable-5.1 copilot=1.0.88
|
Needs a decision, not a merge:
Head: verdict requires_human @f1ebb39, 6/6 checks green, DCO yes, soak 0s of 30m0s. What to do: approve this head ( How to release it:
Clear the cause and the hive re-evaluates on its next pass; the stage moves to |
|
/hive decision approve Approving head On the reviewer question: the scheduled |
|
LGTM label has been added. DetailsGit tree hash: b103bead42a185aadcb2278591e15961d0e52820 |
|
Decision recorded by @clubanderson for head |
|
Supersession sweep found that this PR's claimed issue is already closed by another merged PR.
This PR touches files the merged PR did not, so the hive is resolving it itself: updating head f1ebb39 onto This comment is edited in place by the supersession sweep; it is not duplicated. |
|
New changes are detected. LGTM label has been removed. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
The committed snapshot predates hivecommons/hive#13950 moving the agent runtime to src/crux, so its source links (trajectory, skillreg, manager_pause) 404 on GitHub and the offline fallback of check-internal-links sees stale content. This resyncs docs/content/ mirrors via
npm run sync-hive-docsandnpm run sync-sibling-docs.Fixes #496 (needs-human: the scheduled hive-docs-refresh workflow still has to be added by a maintainer β the hive App cannot push .github/workflows/ files)
β hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88