Skip to content

Explain Forge work through live GitHub conversations - #10

Merged
numinousmuses merged 1 commit into
masterfrom
feat/forge-conversations
Sep 6, 2026
Merged

numinousmuses merged 1 commit into
masterfrom
feat/forge-conversations

Conversation

@numinousmuses

Copy link
Copy Markdown

Connect PR verification to Forge's conversation service. It responds to the contributor's actual change, maintains a current-work comment, and posts meaningful milestones using recorded check results. The updated guide explains contributor PRs, assigned work, and scheduled occurrences.

The workflow keeps verification status reporting. The configured conversation service owns explanatory comments; the earlier status formatter remains available when the feature flag is disabled.

flowchart LR
 subgraph Before
  A[PR update] --> B[Verification task]
  B --> C[Short state template]
 end
 subgraph After
  D[PR update] --> E[Verification task and recorded results]
  E --> F[Contextual explanation and milestone comments]
  F --> G[Maintainer reads current work and evidence in the conversation]
 end
Loading

Validation: documentation checks pass. Platform tests exercise duplicate prevention after lost responses, restart recovery, stale revision rejection, verification identity checks, and explanation-service failure. A live conversation service is running; this PR exercises new CI discovery. Fresh engineering work will demonstrate implementation observations and generated publication after this integration lands.

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown

Numinous Forge · PR verification

Verification passed

The accepted checks passed. Ready for human review.

This run checks the PR; it does not modify code.

Checks passed: Protocol, Documentation.

Revision: 852a4188fcec. Tested merge: 23ae88b05448.

Open workflow and results · Task cb731bcc68e862271982bbadd9b47a73

Last observed 2026-09-06 13:20 UTC. This comment updates as work progresses.

@numinousmuses

Copy link
Copy Markdown
Author

Numinous Forge · Checking the change

This PR wires PR verification into Forge's conversation service so comments respond to the contributor's actual change, keep a single current-work comment updated, and post milestones from recorded check results. The prior short status template stays available behind the feature flag, and verification status reporting is unchanged. Changed files are the two .forge guides and the CI workflow.

Verification is now running on this lane. Two checks are planned: a protocol-behavior check covering duplicate prevention after lost responses, restart recovery, stale-revision rejection, verification identity, and explanation-service failure; and a documentation check for the updated guides. No results are recorded yet, so nothing here is passing—I'll report outcomes once the checks finish.

A live conversation service is running, so this run also exercises new CI discovery. No candidate is present yet. Maintainer decision—review and merge—remains separate and comes after checks report.

Source revision 852a4188fcec. Check results above come from execution records; agent observations are provisional.

This run verifies the PR and does not modify its code.

@numinousmuses

numinousmuses commented Sep 6, 2026

Copy link
Copy Markdown
Author

Numinous Forge · Merged into this fork

This PR has been merged. The recorded checks passed on its reviewed revision.

Next: No further action on this PR.

Verification details
Recorded check Result
Protocol behavior Passed
Documentation Passed

Source revision 852a4188fcec. Verification status comes from execution records; agent observations are provisional.

This run verifies the PR and does not modify its code.

@numinousmuses

Copy link
Copy Markdown
Author

Numinous Forge · Checking the change

Correction to my earlier note: this run does not verify the conversation service, and the protocol check does not cover duplicate prevention, restart recovery, stale-revision rejection, identity checks, or explanation-service failure. Those behaviors come from the PR's own test plan, which is contributor context, not the checks running here.

What this lane actually runs is two Darkbloom checks. The protocol-behavior check exercises provider-message wire encoding, decoding, and protocol regressions; it recorded a pass across 259 tests, including capacity-probe and capacity-quote round trips, error-wire isolation, and the closed-vocabulary failure-code case. The documentation check, which validates freshness, links, and cited paths for the changed guides, is still pending, so nothing about the docs is settled yet.

This is the CI entrypoint, so there is no patch candidate being prepared and none is expected. Passing the protocol check does not establish that the conversation integration works. Review and merge remain a separate human decision after the documentation check reports.

Recorded check Result
Protocol behavior Passed
Documentation Not complete

Source revision 852a4188fcec. Verification status comes from execution records; agent observations are provisional.

This run verifies the PR and does not modify its code.

@numinousmuses

Copy link
Copy Markdown
Author

Numinous Forge · Independent verification passed

Both CI checks have now reported passing on this documentation and workflow change.

The documentation check validated freshness, links, and cited paths for the changed guides (.forge/README.md, .forge/communication.md, and the CI workflow file). It reported a pass. The protocol-behavior check recorded a pass across 259 tests, including capacity-probe and capacity-quote round trips, error-wire isolation, and the closed-vocabulary failure-code case.

Two cautions carry over. The protocol check exercises Darkbloom's provider-message wire encoding and regressions; it does not test Forge's conversation service. The duplicate prevention, restart recovery, stale-revision rejection, identity checks, and explanation-service failure behaviors named in the PR description are contributor test-plan context, not checks that ran here. So these passes confirm the docs are consistent and the protocol suite is green, not that the conversation integration works end to end.

This is the CI entrypoint, so no patch candidate is being prepared. Accepting the behavior and merging remain a human decision.

Recorded check Result
Protocol behavior Passed
Documentation Passed

Source revision 852a4188fcec. Verification status comes from execution records; agent observations are provisional.

This run verifies the PR and does not modify its code.

@numinousmuses
numinousmuses merged commit d178a22 into master Sep 6, 2026
2 checks passed
@numinousmuses numinousmuses added the trigger:ci Forge was triggered by a contributor PR or new commit. label Sep 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trigger:ci Forge was triggered by a contributor PR or new commit.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant