Skip to content

Give Forge one owner for each status comment - #16

Merged
numinousmuses merged 1 commit into
masterfrom
fix/forge-single-writer
Sep 6, 2026
Merged

numinousmuses merged 1 commit into
masterfrom
fix/forge-single-writer

Conversation

@numinousmuses

Copy link
Copy Markdown

Forge now has one writer for each issue or PR status comment. The workflow only submits work and reports commit checks, with read-only PR access. The alternate task reporter and comment finalizer are removed. The comment replaces its current explanation and shows verification details without an appended activity timeline.

This PR installs the workflow change and updates the guides. It also checks for a changed PR head during polling independently of comment reporting, so an obsolete workflow does not hold up the latest revision.

flowchart LR
  subgraph Before
    A[CI workflow] --> B[Optional progress writer]
    C[Task reporter] --> D[Separate task comment]
    E[Conversation service] --> F[Current status with old updates]
  end
  subgraph After
    G[CI workflow] --> H[Task submission and commit checks]
    H --> I[Conversation service]
    I --> J[One edited status comment per issue or PR]
  end
Loading

Validation: platform tests cover routine transitions, publication, merge, new commits, schedule occurrences, lost acknowledgments, ignored legacy milestones, and rejection of comment access from CI. Fresh issue and scheduled runs are being observed from their first update. The configured checks on this PR cover Darkbloom protocol and documentation, not the Forge platform itself.

@numinousmuses

numinousmuses commented Sep 6, 2026

Copy link
Copy Markdown
Author

Numinous Forge · Merged into this fork

Trigger: CI

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 05f6bfd255d9. Verification status comes from execution records; agent observations are provisional.

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

@numinousmuses numinousmuses moved this to Verifying in Darkbloom engineering Sep 6, 2026
@numinousmuses
numinousmuses merged commit 5506650 into master Sep 6, 2026
2 checks passed
@numinousmuses numinousmuses moved this from Verifying to Done in Darkbloom engineering Sep 6, 2026
@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

Development

Successfully merging this pull request may close these issues.

1 participant