Skip to content

docs(dnn): #131 upgrade assessment — 3-phase DNN migration path (#444 α resubmit) - #484

Merged
jsboige merged 1 commit into
masterfrom
docs/444-alpha-upgrade-assessment
Jun 15, 2026
Merged

docs(dnn): #131 upgrade assessment — 3-phase DNN migration path (#444 α resubmit)#484
jsboige merged 1 commit into
masterfrom
docs/444-alpha-upgrade-assessment

Conversation

@jsboige

@jsboige jsboige commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

What

Re-submit the α deliverable from #444 (closed stale): a standalone DNN upgrade assessment doc (docs/dnn/UPGRADE-ASSESSMENT.md, +273 lines) covering the 3-phase migration path DNN 9.11.1 → 9.13.x → 10.x, 2sxc LTS, RazorBlade/RazorEngine/CVE-2021-46703 risks, Stripe/Imageflow/NBrightBuy e-shop inventory, and the current DB/template state.

Why a re-submit

#444 was closed stale — it bundled three independent pieces:

So only α remains useful.

Why a clean branch

The prior split branch (docs/444-alpha-dnn-upgrade-assessment) was based on pre-OWL-merge commit 36c138b7. Diffed against current master c873bcdc, it would have reverted the entire OWL lane as parasites (OwlAdapter.cs fix #481, OwlValidatorLivePathTests.cs #482, UtilityExtensionsLayoutTests.cs #483). This PR was rebuilt fresh on c873bcdc and isolates the single new file — verified: 1 file changed, +273, 0 deletions.

Diff check

 docs/dnn/UPGRADE-ASSESSMENT.md | 273 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 273 insertions(+)

Status

  • Pure doc → no gate, no pipeline impact.
  • Status header inside the doc reads “Draft — awaiting jsboige validation”; @jsboige indicated docs review tomorrow.
  • Verdict QA visuelle = ai-01 / jsboige (I signal, I don't declare PASS).

Related

🤖 Worker po-2023 — dispatch `laj77j` primaire.

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

…α resubmit)

Re-submit the α deliverable from #444 (closed stale) as a clean doc-only PR.
The original #444 bundled α (this assessment) + β (Memo Back loc fix = #446)
+ γ (gpt-5.5 PT task = #447). β and γ are already on master; γ additionally
carried a parasite gpt-5.5→5.4-mini downgrade on 12 sites (avoided). The prior
split branch (docs/444-alpha-dnn-upgrade-assessment) was based on pre-OWL-merge
commit 36c138b and would have reverted #481/#482/#483 as parasites — this PR
isolates the single new file on master c873bcd.

Related: #131 (DNN security/upgrade), #132 (DNN deployment), #134 (release v0.9.0).
Supersedes: #444 (closed stale; β=#446, γ=#447 already merged).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[NanoClaw] LGTM — comprehensive DNN upgrade assessment. Well-structured 3-phase plan with clear blockers (OpenStore .NET 4.8 gates DNN 10.x). Security section accurate (CVE-2025-52488/64095 both critical, patched in 9.13.x). RazorComponent migration inventory complete (12 templates, none in Argumentum app). Execution order pragmatic (security-first → eshop → platform → i18n). Stripe Native decision properly referenced. One note: web.config merge in Phase 1 Step 2 could benefit from a diff tool recommendation (e.g. WinMerge) to reduce manual error risk.

@jsboige
jsboige merged commit 8fbdb09 into master Jun 15, 2026
3 checks passed
@jsboige
jsboige deleted the docs/444-alpha-upgrade-assessment branch June 15, 2026 20:08
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