Repository navigation
Close delivered pm items and keep in progress for active work only - #44
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 metadata-focused sweep reconciles all original in-progress and open records with tracker and merged-PR evidence: seven verified deliveries are closed, stale in-progress claims are removed, three items remain open pending evidence, and the changelog and append-only audit history are updated. No production functions change; validation covers tracker health, history integrity, whitespace/privacy checks, and release checks delegated to CI. Flow diagram for tracker item reconciliationflowchart LR
A["Tracker items"] --> B["Audit tracker records and merged PR evidence"]
B --> C{"Acceptance complete?"}
C -->|Yes| D["Close item and append history"]
C -->|No or ambiguous| E["Keep open"]
B --> F["Stale claim"]
F --> G["Release in-progress status"]
D --> H["Regenerate changelog and validate history"]
E --> H
G --> H
File-Level Changes
Possibly linked issues
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-ado-github-5532dc4b9a65-33: The issue concerns delivery candidate selection owned by pm-github; pm-ado PR 41 changes Azure resolution, while PR 19 implements statement coverage rather than this closeout defect.pm-ado-g4v1: The broad sync epic still needs explicit revisions-to-PM-history reconciliation evidence; related implementation PRs do not prove every criterion.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:
fa7ea79b2d59cf27d7682cb54e7b7052e59e9d2e. The sweep audit remains open with its claim released for orchestrator review. No PR is merged.Summary by Sourcery
Reconcile PM tracker state with delivered work, close verified items, and preserve only genuinely unresolved work as open.
New Features:
Bug Fixes:
Enhancements:
Build:
CI:
Documentation:
Chores: