Re-base the app-bundle overlay onto canonical opencode v1.18.29 (#796) - #797
Merged
Merged
Conversation
…aintained overlay - drift_report.mjs: classify every overlay file against the upstream tree at a tag (added/modified/unchanged + deletion staleness); regenerable, output committed as drift-report.json (#796). - refresh_manifest.mjs: recompute the manifest from the committed overlay vs the upstream base — post-re-base the overlay is maintained against upstream, no longer a byte-exact fork extraction. - drift_gate.mjs: retire the fork-extraction equality checks (invalid after the re-base); keep overlay<->manifest sync and add the upstream archive-sha verification against the materializer's cache stamp.
#796) Materialized the app tree at v1.18.29 (archive sha256 8fd2a4e1... verified against two independent fetches; tag commit 16747470 pinned) and repaired the mechanical breakage class — all packages typecheck clean, unit tests green (app 1063/1063, session-ui 155/155, ui 508/508), vite production build succeeds. Repairs (overlay payload re-applied on the new upstream reality): - marked stack: upstream .29 split the monolith (marked-parser/theme/theme- register). Retire the overlay's fork-era marked.tsx (its nativeParser path was dead); carry the fork's marked-code-span.ts (+test) which the old file imported but the overlay never shipped; re-apply the Amicode payload (KATEX_MACROS, single-dollar math) on the new marked-parser.tsx. - language.tsx: adopt upstream's desktop-native i18n rewrite (plural, RTL, onNativeTranslations); re-apply the L1 default-locale payload (?? "en"). - desktop-native.ts (new overlay): product-name branding (Amicode) now rides the desktop-native label surface upstream's menus consume. - helpers.ts: adopt upstream's compareSessionTime; keep the Amicode recency-window sort + sessionListDirectories (amicode#288). - v2 prompt-input / image-attachments / windows-app-menu: adopt upstream's i18n'd + fileLabel'd versions; re-apply trailingControl (#116) + rounded-sm. - placeholder.ts: retire the overlay's hardcoded 2-arg version (broke upstream's own test); brand via ui/en.ts instead. - message-part.tsx: migrate to the new AnimatedCountList plural-key API + 3-arg typeLabel. - server-session.ts / desktop-menu.ts: adopt upstream wholesale (retired). - session-cache.ts: guard diff_version eviction (fork-side field absent from upstream-shaped stores). - i18n dicts: merge upstream .29's new keys into en + the 17 overlay locales (branded values preserved); overlay upstream's parity test scoped to the overlay's dict set (Amicode keys don't exist in upstream's 28 extra locales). Manifest: upstream_base v1.18.29 @ 16747470, archive sha recorded.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 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. Comment |
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #796.
What lands
ea800b7f…anchors the Phase-0 binary lineage).drift_report.mjs+ committeddrift-report.json, re-runnable): 298 added / 278 modified / 0 unchanged of 576 overlay files; 7 deletions active, 0 stale. The drift gate's fork-extraction equality checks are retired — post-re-base the overlay is maintained against upstream, not extracted from the fork.marked-code-span.ts— a fork file the old overlay never shipped), the desktop-native i18n rewrite (46 locales) with the L1 payload re-applied, i18n dict merges across en + 17 overlay locales, the desktop-native branding surface, and the server-session/session-cache rewrite adoption with the fork-sidediff_versionguard.Honest residuals
enonly; 17 locales hold upstream copy (pre-existing L1 state, recorded follow-up).Phase-0 recon (ledger loop 28a) + full AC evidence in the campaign ledger, loop 28b.