Skip to content

fix(xlsx): exclude phonetic guide text from cell values - #348

Open
y4ho0 wants to merge 1 commit into
iOfficeAI:mainfrom
y4ho0:agent/fix-xlsx-phonetic-text
Open

fix(xlsx): exclude phonetic guide text from cell values#348
y4ho0 wants to merge 1 commit into
iOfficeAI:mainfrom
y4ho0:agent/fix-xlsx-phonetic-text

Conversation

@y4ho0

@y4ho0 y4ho0 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • exclude <rPh> phonetic-guide text from ordinary XLSX cell display values
  • preserve direct shared-string text and rich-text runs
  • keep the guide available separately through the existing format.phonetic readback

Fixes #343

Validation

Using the reporter's attached sample.xlsx:

officecli view sample.xlsx html
# Before: <td data-path="/Sheet1/A1">項目コウモク</td>
# After:  <td data-path="/Sheet1/A1">項目</td>

officecli view sample.xlsx text
# Before: A1=項目コウモク
# After:  A1=項目

officecli get sample.xlsx /Sheet1/A1 --json
# text: "項目"
# format.phonetic: "コウモク"

officecli validate sample.xlsx
# Validation passed: no errors found.

I also created a shared-string cell with two formatted runs (Hello bold, world italic). view text still returns Hello world, view html retains both formatted spans, and officecli validate passes.

Project build:

dotnet build src/officecli/officecli.csproj -c Release --no-restore
# Build succeeded (one pre-existing CS8602 warning in ExcelHandler.SheetShift.cs).

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.

[XLSX] view html renders phonetic guide text as additional cell content

1 participant