add xlsx export suppport to workspace-sheets blueprint

[maxwellpeterson]

update built-in sheets blueprint to support xlsx export

[github-actions]

Preview: pr433-mpeterson-xlsx-export

https://pr433-mpeterson-xlsx-export-router.cloudflare-os-previews.workers.dev

Dashboard · deleted when this PR closes

[ask-bonk]

Findings

• Medium xlsx.js:361: Cells outside declared rows/cols are silently omitted, although the server accepts and persists them and CSV exports them. Export all valid Excel references or reject/prune them when stored.
• Medium xlsx.js:463: nf: "text" converts formulas to literal strings. The app evaluates formulas independently of number formatting, so XLSX export changes workbook behavior.
• Medium xlsx.js:385: Each unmatched apostrophe rescans the remaining formula, producing quadratic work. Valid 8,192-character cells can exceed the 30-second export deadline with only a few hundred formulas. Parse quoted references in one pass.

github run

[ask-bonk]

@maxwellpeterson Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

[ask-bonk]

Submitted one review with 3 actionable inline findings.

github run

[ask-bonk]

Submitted one review with 3 actionable inline findings.

github run

[ask-bonk]

Submitted one review with 2 actionable inline findings. Focused XLSX tests pass (24/24).

github run

[ask-bonk]

Submitted one review with 3 actionable inline findings.

github run

[ask-bonk]

Submitted 1 inline finding. Additional findings:

• High server.js:102: Delayed broadcasts can overwrite a sender’s newer local state, while its own corrective event is ignored.
• Medium xlsx.js:549: =2^3^2 changes from 512 locally to 64 in Excel due to associativity differences.
• Medium xlsx.js:252: Font sizes are stored/rendered as pixels but exported as points, making custom fonts about 33% larger.

github run

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[ask-bonk]

Findings

• Medium xlsx.js:531: Malformed formulas accepted by the grid, such as =SUM(1,2 or ="abc, are emitted as invalid Excel formulas. Validate them or export as text.
• Medium xlsx.js:531: Out-of-grid references remain formulas despite their cells being omitted. For example, =XFE1+1 evaluates locally but becomes #NAME? in Excel.

Focused tests pass (23/23).

github run

[ask-bonk]

Findings

• Medium xlsx.js:456: Names requiring Excel delimiters, such as R1C1 or $Data, remain unquoted, breaking cross-sheet formulas. Quote unchanged names when required.
• Medium xlsx.js:311: Date serial 1 displays 12/31/1899 locally but 01/01/1900 in Excel due to incompatible epochs.
• Medium xlsx.js:303: Integer formatting changes -1.5 from -1 locally to -2 in Excel because rounding semantics differ.

github run

[ask-bonk]

APIError: You do not have access to the organization tied to the API key.

github run

[ask-bonk]

APIError: You do not have access to the organization tied to the API key.

github run

[ask-bonk]

APIError: You do not have access to the organization tied to the API key.

github run