Skip to content

release(3.10.0): promote to master - #664

Closed
vypbot wants to merge 71 commits into
masterfrom
release/3.10.0
Closed

vypbot wants to merge 71 commits into
masterfrom
release/3.10.0

Conversation

@vypbot

@vypbot vypbot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

🚀 Production promotion

Purpose: accept the prepared change in production.

Origin Prepared source Destination Publication
develop@​f5f462b release/3.10.0@​4d7906e master Package status: not published

Ready before review

  • ✅ Build and release validation
  • ✅ Package smoke test
  • ⏳ Protected-branch checks and reviews

After merge

  • After merge, Copilot will tag and publish the accepted production commit.
  • Development reconciliation: develop.

Compare changes · Release control center

Technical details

Operation 63381e09-9a3d-440d-aa28-a8efcaa85308; strategy production-lineage; merge mode auto.

vypbot and others added 30 commits October 20, 2026 23:29
release(3.9.3): reconcile master into develop
…cope: clarify deployment queues and managed pointer hashes
…sted-windows-permission-journal-test-exceeds-jest-timeout

fix: bugfix intermittent hosted Windows permission journal test exceeds Jest timeout (#577)
…bajo-demanda-para-evitar-cuellos-de-botella-en-ci

chore: run RepoWise only on manual dispatch
…-syntax-and-managed-agents-md-hash-scope

docs: clarify deployment queues and managed pointer hashes
…e-analysis-and-complete-bounded-dead-code-and-history-scans

chore: complete bounded manual RepoWise analysis
…d-recovery-for-unsupported-pr-configuration-schemas

fix: diagnose unsupported stored configuration with safe recovery
…lication-callbacks-omit-live-merge-queue-attestations

fix: forward live queue attestations on publication
…-without-an-actionable-issue-diagnostic-or-clarification-card
…-reconciliation-with-opaque-provider-unavailable
@vypbot

vypbot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor Author

Copilot approval · Human review needed

Status: This pull request is outside the configured approval scope.
Completed: 1/1 observed checks succeeded; Bugbot unverified.
Action required: Ask a human reviewer to decide.
Impact: No new approval is confirmed for this revision.
See checks

Technical evidence

Revision: 4d7906e639a7; base: f7f68e699a7f; reason: scope.

@efraespada
efraespada disabled auto-merge October 21, 2026 23:59
@efraespada

Copy link
Copy Markdown
Member

/copilot findings

@efraespada

Copy link
Copy Markdown
Member

Review evidence for the prepared candidate:

This candidate is now frozen by operation 63381e09-9a3d-440d-aa28-a8efcaa85308. Current-head checks and complete findings review remain required before merge.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.74622% with 59 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.13%. Comparing base (f7f68e6) to head (4d7906e).

Files with missing lines Patch % Lines
...c/infrastructure/native_verification_controller.ts 89.53% 5 Missing and 13 partials ⚠️
src/infrastructure/verification_setup_snapshot.ts 83.92% 2 Missing and 7 partials ⚠️
...rc/infrastructure/native_verification_candidate.ts 86.66% 1 Missing and 3 partials ⚠️
src/infrastructure/native_verification_protocol.ts 87.87% 0 Missing and 4 partials ⚠️
src/manager/description/configuration_markers.ts 91.66% 0 Missing and 4 partials ⚠️
src/infrastructure/setup_verification_action.ts 95.16% 0 Missing and 3 partials ⚠️
src/infrastructure/verification_setup_result.ts 90.32% 1 Missing and 2 partials ⚠️
src/infrastructure/verification_setup_runner.ts 95.91% 0 Missing and 2 partials ⚠️
...ication/usecases/actions/initial_setup_workflow.ts 50.00% 0 Missing and 1 partial ⚠️
...ation/usecases/sdd/pre_branch_sdd_gate_use_case.ts 95.83% 1 Missing ⚠️
... and 10 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #664      +/-   ##
==========================================
+ Coverage   94.10%   94.13%   +0.02%     
==========================================
  Files         950      974      +24     
  Lines       30548    31633    +1085     
  Branches     8965     9288     +323     
==========================================
+ Hits        28748    29778    +1030     
- Misses        704      715      +11     
- Partials     1096     1140      +44     
Files with missing lines Coverage Δ
src/actions/github_action.ts 95.72% <100.00%> (+0.23%) ⬆️
src/actions/github_action_execution.ts 93.75% <ø> (ø)
src/actions/local_action_configuration_sections.ts 86.00% <ø> (ø)
src/actions/local_action_execution.ts 92.30% <ø> (ø)
src/application/contracts/input_keys.ts 100.00% <ø> (ø)
src/application/contracts/setup_documentation.ts 100.00% <ø> (ø)
src/application/errors/bugbot_mutation_error.ts 100.00% <100.00%> (ø)
...tion/policies/application_error_message_catalog.ts 100.00% <100.00%> (ø)
.../policies/application_error_presentation_policy.ts 100.00% <100.00%> (ø)
src/application/policies/bugbot_message_catalog.ts 100.00% <100.00%> (ø)
... and 90 more

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vypbot vypbot left a comment •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Current status: 6 findings originating in this review require attention.
Last reconciled on 4d7906e. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 6 potential problems when commit 4d7906e was analyzed. This snapshot is historical; use the status block above for current state. 6 findings are linked to changed code.

Findings

  • high: Invalid action reference prevents the comment workflow from loading — .github/workflows/copilot_comment_runtime.yml:83
  • medium: Source workflow uses an invalid local reusable-workflow reference — src/domain/copilot_comment_workflow.ts:3
  • medium: Cleanup can delete a file under an unvalidated session path — src/infrastructure/verification_setup_runner.ts:58
  • low: Native fixture leaves a generated runtime file behind — src/tooling/__tests__/fixtures/verification_native_fixture.ts:60
  • low: Reusable workflow inspection drops the source-verification override — src/infrastructure/setup_management_workspace_adapter.ts:101
  • low: Reusable workflow is both required and out of scope — specs/repository-verification-setup-hooks.md:22

Comment thread src/domain/copilot_comment_workflow.ts Resolved
@vypbot

vypbot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

@efraespada

Copy link
Copy Markdown
Member

/copilot dismiss "copilot_comment_runtime.yml:83:invalid-local-action-reference" "src/domain/copilot_comment_workflow.ts:3:invalid-local-reusable-workflow-reference" "src/tooling/tests/fixtures/verification_native_fixture.ts:60:leaves-fixture-runtime" "setup_management_workspace_adapter:101:prepare-source-verification-omitted"

@vypbot

vypbot commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

4 findings were dismissed.

@efraespada

Copy link
Copy Markdown
Member

Superseded without merge or publication. Complete review found two actionable corrections: ownership-safe native request cleanup and the SDD reusable-workflow scope clarification. The immutable prepared source of operation 63381e09-9a3d-440d-aa28-a8efcaa85308 will remain unchanged. A fresh unpublished release attempt will contain the corrections and repeat all current-head checks and findings review. Four false positives have evidence replies and resolved threads; the real findings remain historical until their correction is linked.

@efraespada

Copy link
Copy Markdown
Member

Both actionable findings have been delivered through accepted production promotion #668 at 9e505f5. Commit d33064a restricts request cleanup to validated requests created by the invocation, adds three reproduced regressions and success/failure cleanup assertions, and clarifies consumer-hook versus central-coordinator scope. All 93 native tests and the final CI/platform gates passed. This superseded PR's original source remains unchanged for audit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants