Repository navigation
Reconcile VSCode-Server-DotNetCore in the Hub After Its Resync - #3126
Conversation
Delete the stale driftNotes entry, tick the three adoption items on live main evidence, regenerate the workflow-reuse report, and refresh the audit report. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…t Item Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
A recorded local strict-review pass covers head |
There was a problem hiding this comment.
🔵 Needs a closer look
Two review findings remain unresolved in the workflow documentation and audit evidence.
0 open findings
What changed in this PR
Reconciles hub metadata and audit reports with the resynced VSCode-Server-DotNetCore repository.
Changes:
- Removes stale registry drift metadata.
- Records completed workflow adoptions.
- Regenerates workflow reuse and audit reports.
| File | Summary |
|---|---|
reports/workflow-reuse.md |
Updates fleet workflow metrics. |
reports/vscode-server-dotnetcore/audit.md |
Records current audit results; add file:line evidence for the drift finding. |
registry/repos.json |
Removes the obsolete drift note. |
docs/reusable-workflows.md |
Records adoptions; reconcile the Stage 3 carrier count and checklist. |
🧠 Review effort: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #3126 +/- ##
==========================================
Coverage ? 63.92%
==========================================
Files ? 17
Lines ? 9145
Branches ? 0
==========================================
Hits ? 5846
Misses ? 3299
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
A recorded local strict-review pass covers head |
|
Both findings in the round on 628254f are fixed in 3a2c1c5 and e0d49d9: the Stage 3 carrier count is dropped from the adoption line (a count in a living document goes stale, and the generated workflow-reuse report carries the live figure), and the audit drift finding now cites its evidence, the audit run and the carried GOVERNANCE.md section line numbers. |
… Fixes to Main (#3146) - #3135 makes `gh-write-guard` read a gh write's subcommand and target the way gh parses them: the subcommand is found past flags before the command and before the subcommand, an `R` inside a short-option cluster is read as a target, `GH_REPO` is read as a target, a run-time repository owner on a write fails closed, and a run-time word where `pr` or `merge` goes fails closed beside `--admin` or where word splitting may split it - #3139 releases the bootstrap lock when a `bootstrap.ps1` run ends rather than when its process does, so a second run typed at the same pwsh prompt is no longer refused the lock - #3130 matches tool names and the `all` keyword by case in the Windows agents loaders, as the Linux loaders already do - #3128 reads CodeRabbit's rate limit on a reply to an explicit review command in `pr_review.py` - #3126 reconciles VSCode-Server-DotNetCore in the hub after its resync - #3124 logs the Docker task's build-base leg in to Docker Hub - #3122 names the carried skill and the skipped review correctly - #3120 states the subprocess encoding rule in `CODESTYLE.md`, where carriers read Python style - #3118 and #3111 move the release-guard tests' workflow-command checks onto `runner_commands`, and test the alternatives wiring at the `audit_repo` and `fidelity_pass` call sites - #3116 stops the doc gates' printed names forging workflow commands - #3113 adds a catalog snippet for the release-with-smoke caller - #3109 moves the `workflow_reuse` fetch onto `audit.contents_text` - #3105 accepts `.codecov.yml` as an alternative to `codecov.yml` - #3103 adds a pip ecosystem to the carried Dependabot snippet - #3100 prefers the venv's ruff and pyright for a declared pip directory - #3098 tracks composite action pins in Dependabot - #3096 reads an uppercase Markdown suffix as Markdown at every prose gate branch After this merges, the host-safety installer payload changes, so the maintainer runs the installer after promotion. Closes #1416 Closes #1425 Closes #1578 Closes #1874 Closes #1892 Closes #1899 Closes #2804 Closes #2850 Closes #2852 Closes #2854 Closes #2996 Closes #3065 Closes #3067 Closes #3069 Closes #3079 Closes #3080 Closes #3089 Closes #3093 Closes #3101 Closes #3106 Closes #3107 Closes #3137 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added a pull-request smoke build that runs when relevant release files change and reports the combined validation and build status. * Repository audits now recognize `.codecov.yml` as an alternative to `codecov.yml`. * **Improvements** * Python linting and type checks use tools from declared project environments when available, with fallbacks for missing tools. * Workflow and lint reports safely display unusual filenames and command-like text. * GitHub write-target checks now account for additional command forms and fail closed when targets cannot be verified. * Rate-limit detection now recognizes another CodeRabbit response format. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Reconciles the hub's records for VSCode-Server-DotNetCore after its resync, each change checked against the live
mainof that repository.driftNotesentry inregistry/repos.json, sincemaincalls the hub'spublish-docker-readme-task.ymland keeps no local copy.get-version-task.yml, Stage 4 "vanilla Docker only", and Stage 5publish-docker-readme-task.ymlitems indocs/reusable-workflows.md, and drops adelete-branch: truequalifier from the merge-bot item thatmaindoes not set.reports/workflow-reuse.md.reports/vscode-server-dotnetcore/audit.mdfrom aspec/audit.pyrun at hubddaeefb4againstmain, 0 defect, letter, or error findings.Local strict review: two rounds, all findings fixed.
Closes on promotion: #1899
🤖 Generated with Claude Code