Skip to content

Reconcile VSCode-Server-DotNetCore in the Hub After Its Resync - #3126

Merged
ptr727 merged 5 commits into
developfrom
feature/auto-1899
Oct 16, 2026
Merged

ptr727 merged 5 commits into
developfrom
feature/auto-1899

Conversation

@ptr727

@ptr727 ptr727 commented Oct 16, 2026

Copy link
Copy Markdown
Owner

Reconciles the hub's records for VSCode-Server-DotNetCore after its resync, each change checked against the live main of that repository.

  • Deletes the stale driftNotes entry in registry/repos.json, since main calls the hub's publish-docker-readme-task.yml and keeps no local copy.
  • Ticks the Stage 3 get-version-task.yml, Stage 4 "vanilla Docker only", and Stage 5 publish-docker-readme-task.yml items in docs/reusable-workflows.md, and drops a delete-branch: true qualifier from the merge-bot item that main does not set.
  • Regenerates reports/workflow-reuse.md.
  • Rewrites reports/vscode-server-dotnetcore/audit.md from a spec/audit.py run at hub ddaeefb4 against main, 0 defect, letter, or error findings.

Local strict review: two rounds, all findings fixed.

Closes on promotion: #1899

🤖 Generated with Claude Code

ptr727 and others added 3 commits October 15, 2026 23:42
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>
Copilot AI lite review requested due to automatic review settings October 16, 2026 06:50
@coderabbitai

coderabbitai Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 98a7e516-8597-4061-8b76-0abb2dea88b7
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ptr727

ptr727 commented Oct 16, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head 628254f3eda695fc9f4751c685bf9d3e00b382a4, the content this pull request carries at that commit against develop, and it recorded 5 findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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

codecov Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@ddaeefb). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #3126   +/-   ##
==========================================
  Coverage           ?   63.92%           
==========================================
  Files              ?       17           
  Lines              ?     9145           
  Branches           ?        0           
==========================================
  Hits               ?     5846           
  Misses             ?     3299           
  Partials           ?        0           
Flag Coverage Δ
python-3.13 63.92% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@ptr727

ptr727 commented Oct 16, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head e0d49d95f9dd104fb5cbfeb4e67093378f810336, the content this pull request carries at that commit against develop, and it recorded 2 findings.

@ptr727

ptr727 commented Oct 16, 2026

Copy link
Copy Markdown
Owner Author

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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No unresolved review issues were identified.

0 open findings

🧠 Review effort: Lite

@ptr727
ptr727 merged commit 40a131c into develop Oct 16, 2026
14 checks passed
@ptr727
ptr727 deleted the feature/auto-1899 branch October 16, 2026 07:11
ptr727 added a commit that referenced this pull request Oct 16, 2026
… 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 -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants