[#374] 🤖 - Localize and simplify deployment presentation - #374
Conversation
|
To preview the documentation for this pull request, visit the following URL:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #374 +/- ##
===========================================
+ Coverage 90.94% 92.44% +1.49%
===========================================
Files 135 644 +509
Lines 6251 16644 +10393
Branches 1338 4258 +2920
===========================================
+ Hits 5685 15386 +9701
- Misses 172 505 +333
- Partials 394 753 +359
🚀 New features to boost your workflow:
|
Bugbot: review incomplete
Incomplete coverage
|
There was a problem hiding this comment.
Current status: Overall coverage is partial; this snapshot does not prove that all of its findings are resolved.
Last reconciled onc6862c5. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit 519de5d was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- medium: Publish the sanitized failure message in milestone comments —
src/application/services/deployment_orchestration_runtime.ts:70
📌 Summary
Localizes deployment orchestration across issue dashboards, milestone notifications, managed pull requests, and GitHub Actions Job Summaries. Adds locale-aware catalogs, safe BCP-47 resolution, atomic English fallback, durable locale snapshots, and compatibility for legacy version-1 operation state.
🎯 Related Issues / Tickets
🧩 Scope of Changes
en-USandes-ESsupport.repository-localeand preserve inherited issue and pull-request locale overrides.Result.steps.🛠️ Technical Details
🔍 How to Test
pnpm testandpnpm run test:coverage.pnpm run typecheckandpnpm run lint.pnpm run validate:documentation,pnpm run validate:workflows,pnpm run validate:specifications,pnpm run validate:npm-package,pnpm run validate:build, andpnpm run smoke:npm-package.repository-localeset toen-USores-ES, and verify that the issue dashboard, milestone messages, managed PR, and Job Summary use their configured destination locale.fr-FRand verify that a safe catalog is resolved or the complete artifact falls back to English.🧪 Test Coverage
📸 Screenshots / Recordings (UI changes only)
Not included. The change affects generated GitHub issue, pull-request, milestone, and Job Summary content rather than a standalone visual interface.
🚀 Deployment Notes
Details: Existing version-1 deployment state remains compatible. New deployments snapshot the effective locale profile automatically, and generated bundles are included in the change. Locale Variables remain optional because issue and pull-request overrides inherit
repository-localewhen empty.🔒 Security Considerations
Dynamic catalog output is validated before Markdown publication, including rejection of unsafe markup, URLs, mentions, commands, bidi controls, control characters, and table delimiters. Provider facts and untrusted values remain subject to bounded rendering and URL validation.
📈 Performance Impact
Catalog resolution is cached once per destination. The additional resolution work is bounded and does not materially change the performance characteristics of supported locales.
📝 Notes for Reviewers
Please review the locale-scope mapping carefully: issue surfaces use the effective issue locale, managed PRs use the effective pull-request locale, and Job Summaries use the repository locale. Also verify atomic fallback behavior, compatibility with legacy version-1 operations, and removal of internal step-log replay from operator-facing output.
✅ Checklist
📚 Additional Context
The change updates the deployment orchestration specification and repository localization evidence, adds coverage budgets for the new presentation behavior, and includes regenerated action, CLI, and API bundles.