Repository navigation
Close delivered pm items and keep in progress for active work only - #104
Merged
Merged
Conversation
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis sweep audits every original in-progress and open item against merged ancestry, acceptance criteria, and PR attribution, closing five verified deliveries, reopening or retaining incomplete work, adding one audited open chore, and recording all lifecycle decisions in append-only history; it also updates the changelog for the verified deliveries. No production code changes. Sequence diagram for evidence-based tracker status changessequenceDiagram
participant Sweep as Audit sweep
participant Tracker as Tracker records
participant PRs as Merged and open PR evidence
participant History as Append-only history
participant Changelog as CHANGELOG.md
Sweep->>Tracker: Read in-progress and open items
Sweep->>PRs: Check merged ancestry, acceptance, and attribution
PRs-->>Sweep: Return delivery evidence and gaps
alt Verified delivery
Sweep->>Tracker: Set status to closed
Sweep->>Changelog: Add verified delivery entry
else Incomplete or unverified delivery
Sweep->>Tracker: Set or retain status as open
end
Sweep->>History: Record lifecycle decision
State diagram for audited work-item lifecyclestateDiagram-v2
[*] --> in_progress
in_progress --> closed: verified merged delivery
in_progress --> open: incomplete or unverified acceptance
open --> closed: later verified delivery
open --> open: retained or claim released
closed --> closed: historical status retained
Flow diagram for the tracker status auditflowchart TD
A[Audit in-progress and open items] --> B{Complete acceptance and merged PR evidence?}
B -->|Yes| C[Close item]
B -->|No| D{Active work established?}
D -->|Yes| E[Keep open]
D -->|No| F[Release claim and keep or return open]
C --> G[Record append-only history]
E --> G
F --> G
G --> H[Update changelog for verified deliveries]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
The in-progress sweep closed delivered items with completed_at set to the reconciliation time. Analytics that read completed_at as the delivery time (cycle time, recall utility, burndown) then see work finish up to weeks late. Each closure now carries the cited PR's merge time as completed_at; closed_at keeps the tracker reconciliation time. Recorded with pm update --completed-at, so the correction is an appended history event.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Owner
Author
|
Orchestrator review of this sweep (done by hand; review bots here are rate-limited or not enabled):
Merging once CI is green on this head. |
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.
Close verified deliveries and keep in progress reserved for active work. Every original in-progress and open item was checked against full tracker records and merged/open PR claims, comments and acceptance criteria. Incomplete or ambiguous work remains open. Current stale assignments are released; historical claim identities remain provenance.
Retained open after review:
pm-rl-apvf: Returned to open in the 2026-10-24 sweep: Open PR Refuse recursive tracker proposals without paired baseline wins #95 carries remaining recursive-promotion work; multiple narrower merged implementations do not establish all execution, contamination, crash and resource criteria in one verified delivery. No active agent is established.pm-rl-warf: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-longidlock: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-rsipower: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-holderprobe: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-lmresource: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-readusage: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-lintruntime: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-9uz4: Returned to open in the 2026-10-24 sweep: Merged PR 96 verified eight closeouts and clean isolated health, but legacy provenance and environment advisories remain; open PR Refuse recursive tracker proposals without paired baseline wins #95 still references this health work. No active agent is established.pm-rl-corpusnofollow: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-streamtiming: Returned to open in the 2026-10-24 sweep: Recorded receipts show a deterministic descriptor-close fixture, but no merged PR title/body/commit names this item and its comments do not identify the delivering PR. Attribution remains unverified; no active work.pm-rl-nogitdata: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-processcancel: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-stdin: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-7c7m: No merged implementation establishes completion of the recorded scope; retain the existing open work.pm-rl-8nev: Returned to open in the 2026-10-24 sweep: Recorded receipts show an improved running-state barrier, but no merged PR claim or item-comment delivery link establishes attribution for this record. Retain the real-container acceptance for review; no active work.pm-rl-7073: No merged implementation establishes completion of the recorded scope; retain the existing open work.Validation: package changelog generation/check, pinned strict tracker health with required merge drivers under isolated Git identity, changed-item history verification, independent original-history prefix checks, linked
git diff --check, and public added-text/scope checks passed. Existing history records are unchanged. No production function changed, so source-function revert proof is inapplicable; mainline ancestry, acceptance and lifecycle/history checks provide the evidence.Where strict health required it, the repository-pinned
pm merge installregenerated the extension merge fence. Existing historical advisories are retained; no history normalization or repair was used. The pm-vcs text self-host archive is regenerated and independently checked when applicable.The PR CI runs complete
npm run release:checkandbun run release:check(nativejust release-checkfor Rust). Final exact-head CI results are reported separately; pending, skipped and neutral review checks are not approval.Commit:
c106c8f9563b62fe4dac31b4a68a108120adf493. The sweep audit remains open with its claim released for orchestrator review. No PR is merged.Summary by Sourcery
Reconcile project tracker statuses with verified delivery evidence while preserving provenance and keeping only actively established work in progress.
New Features:
Enhancements:
Documentation:
Tests:
Chores: