[SuruchBoss]

What this changes

Closes #59, the last of the sprint-final queue. This takes the fix route, not the README-limit route. The fix is the small one #59 itself suggests: carry each series' source column in ChartSeries, with a test that fails on main.

The bug: chartDataFrom skips a column with no numbers, so a text column between the labels and the numbers is not a series. seriesRefsFrom built each reference from the series' position (startCol + labelOffset + index), not from its column. With Branch | Notes | Q1 | Q2, the exported chart therefore referred to Notes and Q1. The chart in the app was right, and the cached values in the file were right too. But Excel re-reads the references, so it drew the wrong data.

The fix:

• ChartSeries gains column: the column it was read from, counted from the range's first column. chartDataFrom fills it in.
• seriesRefsFrom builds the name and value references from that column. The same goes for a pie's chosen series, which used seriesIndex as a column offset in the same way.
• Nothing else reads the field. The on-screen chart and the PDF draw from points, as before.

Reproduced on main first

The steps are the issue's: Branch | Notes | Q1 | Q2 with 4 rows, A1:D5, bar chart, export.

	main	this PR
Q1 series	Sheet1!$B$1, $B$2:$B$5 (Notes)	Sheet1!$C$1, $C$2:$C$5
Q2 series	Sheet1!$C$1, $C$2:$C$5 (Q1)	Sheet1!$D$1, $D$2:$D$5
Pie of the second series	$C$2:$C$5 (Q1)	$D$2:$D$5 (Q2)
Categories	$A$2:$A$5	$A$2:$A$5

How it was verified

• npm run verify (all ten gates): all ten gates green on d09e01c. The current head 6f54079 carries that tree unchanged; it only adds the merges of เซลล์รูปแบบเวลาสะสม [h]:mm โชว์ชั่วโมงของวันแทนชั่วโมงรวม · Elapsed-time formats like [h]:mm show the hour of the day #219's head and then of main 7c2bb7f (Show zero and negative date serials as Excel does (#219) #239).

Gate	Result
lint · check:readme · check:screens	pass
check:deps	every advisory fixed or accounted for
test	2025 passed (2014 by the README's static count)
check:mutants	32/32 killed
build · check:bundle	3804/4200 KB, largest chunk 934/1300 KB
check:a11y	121 checks passed
check:e2e	60 flows passed

• New test in xlsxCharts.test.ts: "points each series at its own column when a text column sits between them ([area:charts] กราฟในไฟล์ .xlsx ที่ส่งออกชี้คอลัมน์ผิด เมื่อมีคอลัมน์ข้อความคั่น · Charts in the exported .xlsx point at the wrong columns (sev:high) #59)". It exports the issue's table with a bar chart and a pie of the second series, opens the real .xlsx, and reads xl/charts/chart1.xml and chart2.xml.
• Re-checked against the old code: with main's xlsxChartXml.ts, the new test fails (no $D$1).
• The seriesRefsFrom and valueExtent fixtures gain the new field. Nothing else in them changes.
• READMEs, both languages: a sentence after "a chart in the exported .xlsx is a real chart", saying each series refers to the column it was read from, and what went wrong before.
• Recounted: 2014 tests in 145 files, after merging เซลล์รูปแบบเวลาสะสม [h]:mm โชว์ชั่วโมงของวันแทนชั่วโมงรวม · Elapsed-time formats like [h]:mm show the hour of the day #219. Picture 27 retaken in both languages and checked by eye, blessed after the retake. Social card rebuilt and checked by eye.

The branch

Head: claude/new-session-m6mehm-59, a second branch the owner allowed so #219 and #59 could be open together. It was opened stacked on PR #239; now that #239 is in main, its base is main, and the merge of main was pushed after the base changed so CI and CodeQL run against main.

🤖 Generated with Claude Code

https://claude.ai/code/session_013pmWZsbmugfpqtYazrzfjj

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
excel-to-go	Ready	Preview	Oct 5, 2026 4:57am UTC

[SuruchBoss]

PO review: merged at 6f54079. The merge commit is 726af92.

• npm run verify: all ten gates green on 37f87a2. That head has the same tree as 6f54079 (f9e0736), so the run counts for the merged head. 2025 tests passed, mutants 32/32, a11y 121 checks, e2e 60 flows.
• CI: green on 6f54079.

I tried it as a user on a production build. I typed the issue's table, Branch | Notes | Q1 | Q2 with 4 rows, selected A1:D5, and made a bar chart and a pie. I set the pie to Q2, then exported and unzipped the file:

Chart	Name	Values	Categories
chart1.xml bar, Q1	Sheet1!$C$1	$C$2:$C$5	$A$2:$A$5
chart1.xml bar, Q2	$D$1	$D$2:$D$5	$A$2:$A$5
chart2.xml pie	$D$1	$D$2:$D$5	$A$2:$A$5

• Nothing points at the Notes column (B).
• The on-screen bar and pie match the table.
• The exported file opens back in the app.
• There were no page errors or alerts at 1280, and none on load at 390.

After this merge, the hold set has one ticket left: #241. Opening an .xlsx turns 0 and negative date/time cells into text.

---

Generated by Claude Code