Skip to content

Localize Checks and branch sync feedback - #388

Merged
efraespada merged 3 commits into
developfrom
codex/localized-check-and-command-feedback
Sep 15, 2026
Merged

efraespada merged 3 commits into
developfrom
codex/localized-check-and-command-feedback

Conversation

@efraespada

@efraespada efraespada commented Sep 15, 2026

Copy link
Copy Markdown
Member

This PR localizes Check titles and summaries to the effective surface locale while preserving stable Check names, and adds concise localized feedback for explicit branch-sync outcomes. It also strengthens replay-safe publication and legacy English defaults.

What changed

  • Localizes human-facing Check titles and summaries for issue, pull request, and repository surfaces while keeping names such as Copilot / Review stable.
  • Publishes source-correlated sync-branch replies for aligned, dry-run, and successful merge outcomes without exposing internal execution steps.
  • Adds bounded, sanitized rendering for localized Check titles and branch references.
  • Expands regression coverage and documents locale behavior, branch-sync feedback, and compatibility expectations.

Review notes

  • Stable Check names remain unchanged for branch protection; only human-facing titles and summaries are localized.
  • Branch-sync replies are emitted only for successful SyncBranchUseCase results tied to an explicit comment: correlation, with malformed merge successes rejected.

@docs-page

docs-page Bot commented Sep 15, 2026

Copy link
Copy Markdown

To preview the documentation for this pull request, visit the following URL:

docs.page/vypdev/copilot~388

Documentation is deployed and generated using docs.page

@vypbot
vypbot self-requested a review September 15, 2026 10:01
@vypbot vypbot added this to vypdev Sep 15, 2026
@vypbot vypbot moved this to In Progress in vypdev Sep 15, 2026
@codecov-commenter

codecov-commenter commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.72131% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.03%. Comparing base (274db40) to head (a4db198).
⚠️ Report is 775 commits behind head on develop.

Files with missing lines Patch % Lines
src/actions/github_action_completion.ts 94.73% 0 Missing and 1 partial ⚠️
...ion/policies/semantic_result_publication_policy.ts 96.55% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #388      +/-   ##
===========================================
+ Coverage    90.94%   93.03%   +2.08%     
===========================================
  Files          135      656     +521     
  Lines         6251    17486   +11235     
  Branches      1338     4503    +3165     
===========================================
+ Hits          5685    16268   +10583     
- Misses         172      471     +299     
- Partials       394      747     +353     
Files with missing lines Coverage Δ
...ication/policies/action_summary_message_catalog.ts 100.00% <ø> (ø)
...rc/application/policies/copilot_evidence_policy.ts 100.00% <100.00%> (ø)
...pplication/policies/publication_message_catalog.ts 100.00% <ø> (ø)
src/actions/github_action_completion.ts 95.86% <94.73%> (ø)
...ion/policies/semantic_result_publication_policy.ts 98.17% <96.55%> (ø)

... and 713 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 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Bugbot: review complete

Current status: No active findings on a4db198.

Pull request · Verified commit · Workflow run

@efraespada
efraespada merged commit b1c9ca3 into develop Sep 15, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in vypdev Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants