Skip to content

wise-crane-209 - #963

Closed
briansrls wants to merge 2 commits into
mainfrom
session/wise-crane-209
Closed

briansrls wants to merge 2 commits into
mainfrom
session/wise-crane-209

Conversation

@briansrls

Copy link
Copy Markdown
Contributor

Opened from session-dashboard for session wise-crane-209.

briansrls and others added 2 commits April 27, 2026 03:35
Closes the #920 post-merge citation gap. Header now states the file
is sourced from UCD 15.x (UAX #11 East Asian Width) for the display-
width tables and is intentionally 15.x compatible rather than pinned
to a specific minor. UAX #9 is explicitly not consulted (no bidi).
No behavior changes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Codex-connector inline on #954 flagged that the original "additions
fall to Narrow by default" wording was misleading: char_display_width
checks wide_blocks / zero_width_blocks first, so newly assigned code
points inside an already-enumerated range inherit that block's class,
not Narrow. Only code points outside every enumerated range fall to
Narrow. Reworded to spell that out, and noted that range-adding (not
just range-filling) UCD minors require revisiting the tables.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@briansrls

Copy link
Copy Markdown
Contributor Author

Closing as duplicate per parent (tidy-dove-734) explicit guidance at #941: aggregate #949 is the intended integration path; per-worker W-C2 PRs stay closed until Director explicitly switches strategy. Fix commit 0f5b81568 lives on session/wise-crane-209 for cherry-pick onto #949.

@briansrls briansrls closed this Apr 27, 2026
@briansrls

Copy link
Copy Markdown
Contributor Author

Duplicate of the aggregate cleanup path. W-C2 is already integrated into PR #949 as d709f01bd, and the later UAX #11 wording correction is in #949 as a21b0d62c.

Please keep this draft PR closed/duplicate unless Director explicitly switches from the manager aggregate to per-worker PRs. The aggregate is the current merge path.

— sent from tidy-dove-734 (inbox #941); reply at #941

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.

1 participant