Add Excel design reconstruction skill 🤖🤖🤖

[furukawa658]

Adds a reproducible workflow for converting layout-dependent Excel designs into traceable machine-readable tables. Includes mapping guidance, OOXML validation, a reconstruction script, and tests.

Validation:

• 34 tests passed on openpyxl 3.1.4 and 3.1.5
• npm run skill:validate: 430 skills valid
• npm run build: passed

Business approval: Furukawa658

[github-actions]

🔒 PR Risk Scan Results

Scanned 5 changed file(s).

Severity	Count
🔴 High	0
🟠 Medium	0
ℹ️ Info	2

Severity	Rule	File	Line	Match
ℹ️	skill-script-touched	skills/excel-design-reconstruction/scripts/reconstruct.py	1	skills/excel-design-reconstruction/scripts/reconstruct.py
ℹ️	skill-script-touched	skills/excel-design-reconstruction/tests/test_reconstruction.py	1	skills/excel-design-reconstruction/tests/test_reconstruction.py

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

[github-actions]

🚦 Submission status: 👀 Ready for review

Risk tier: merge-risk:high — Privileged execution, automation, or review-policy change
Required to merge: passing submission-gate checks plus 1 approval from reviewers with write access, including a maintainer with admin or maintain permission.

Why this tier

• skills/excel-design-reconstruction/scripts/reconstruct.py is a high-risk path (automation, scripts, MCP config, hooks, or review policy)
• skills/excel-design-reconstruction/tests/test_reconstruction.py is a high-risk path (automation, scripts, MCP config, hooks, or review policy)

Automated checks

Check	Status	Details
Line endings	✅ Passed	Passed · logs
Spelling	✅ Passed	Passed · logs
Generated README consistency	✅ Passed	Passed · logs
Skill validation	✅ Passed	Passed · logs
Skill lint (vally)	✅ Passed	Passed · logs
Risk scan	✅ Passed	Passed · logs
Contributor reputation	✅ Passed	Passed · logs
Duplicate resource scan	✅ Passed	Passed · logs
PR quality signal	⏭️ Skipped	Skipped by its workflow · logs

Review

• Approvals: 0/1
• Assigned reviewer: not assigned yet — comment /request-review to ask for one
• Review target date: not set
• Still needed: 1 more approval(s); an approval from a maintainer with admin or maintain permission
• The core-maintainers pool is not staffed yet; an approver with admin or maintain permission is required instead.

Commands

Command	Who	What it does
/rerun-checks	PR author, maintainers	Re-runs failed or incomplete checks and re-evaluates this gate
/request-review	PR author, maintainers	Asks the review rotation to assign a reviewer (adds needs-reviewer)

_{Updated for a8b647c · gate run · This comment is maintained automatically — see submission gate docs.}

[github-actions]

🔍 Vally Lint Results

✅ All checks passed

Scope	Checked
Skills	1
Agents	0
Total	1

Severity	Count
❌ Errors	0
⚠️ Warnings	0
ℹ️ Advisories	0

Summary

Level	Finding
ℹ️	✅ excel-design-reconstruction (2/2 checks passed)
ℹ️	✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️	✓ spec-compliance: All spec checks passed.
ℹ️	✓ [valid-refs] All file references across 1 skill(s) are valid.
ℹ️	✓ valid-refs: All file references resolve to existing files within the skill directory.
ℹ️	1 skill(s) linted, 1 passed

Full linter output

### Linting skills/excel-design-reconstruction
✅ excel-design-reconstruction (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

[Copilot]

🟡 Changes recommended

Date extraction can alter stored values, and verification does not authenticate the complete workbook package.

5 open findings

• Workbook integrity verification trusts an editable manifest hash · New
• Date cells are converted instead of preserving raw OOXML values · New
• Missing column-count guard permits ranges beyond Excel's XFD limit · New
• Skill frontmatter description must use single-quoted YAML · New
• Routing references unavailable excel-design-review-evidence skill · New

What changed in this PR

Adds a deterministic skill for reconstructing layout-dependent Excel documents into traceable tables.

Knowledge uplift: High.

Changes:

• Adds explicit mapping, approval, reconstruction, and verification workflows.
• Adds OOXML safeguards and 34 focused tests.
• Documents schemas, outputs, and dependencies.

File	Description
skills/​excel-design-reconstruction/​SKILL.md	Defines the reconstruction workflow.
skills/​excel-design-reconstruction/​scripts/​reconstruct.py	Implements inspection, generation, and verification.
skills/​excel-design-reconstruction/​tests/​test_reconstruction.py	Tests schemas, security controls, and tampering.
skills/​excel-design-reconstruction/​references/​mapping.md	Documents mapping and output contracts.
skills/​excel-design-reconstruction/​requirements.txt	Pins the supported openpyxl range.
docs/​README.skills.md	Registers the new skill.

🧠 Review effort: Balanced

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[furukawa658]

Addressed the five Copilot review findings and added the English-language skill update in 4863262: canonical regenerated OOXML package comparison (excluding ZIP encoding/timestamps and core created/modified timestamps); raw numeric/date OOXML values retained; 16,384-column guard including row_key; single-quoted YAML description; unavailable skill references removed. The source workbook's text and approved headers remain unchanged.

Validation: 38 tests passed on openpyxl 3.1.4 and 3.1.5; all 430 skills passed validation; npm run build and git diff --check passed; the synthetic packet and English text/link checks passed. Independent review of the English instructions is still pending.

[Copilot]

🟡 Changes recommended

The core workflow omits inspection and mapping steps, and several reproducibility and validation gaps remain.

4 open findings

• Validate user-defined column headers before workbook creation · New
• Record and verify the build-time openpyxl version · New
• Add explicit inspection and mapping steps to the workflow · New
• Remove image inspection from the workbook-only workflow · New

5 resolved since last review

• Workbook integrity verification trusts an editable manifest hash
• Missing column-count guard permits ranges beyond Excel's XFD limit
• Date cells are converted instead of preserving raw OOXML values
• Routing references unavailable excel-design-review-evidence skill
• Skill frontmatter description must use single-quoted YAML

🧠 Review effort: Balanced

[furukawa658]

Addressed all four Copilot review findings in 2a23476:

• Added explicit Step 1 (inspect), Step 2 (mapping.json definition linking references/mapping.md), and Step 3 (fingerprint and approval) sections before Step 4 (build) in SKILL.md.
• Updated references/mapping.md to say inspect the source workbook instead of inspect the source image.
• Applied validate_text (Excel 32,767-character limit and disallowed control character check) to user-defined column names in prepare_tables and build_packet.
• Recorded openpyxl_version in manifest.json during build and verified it before package comparison in verify_packet with a clear version mismatch error.

Validation: 40 unit tests pass, npm run skill:validate and npm run build pass with clean LF line endings.

/request-review

[Copilot]

🟡 Changes recommended

Print-area parsing and nondeterministic formula serialization can prevent valid workbooks from completing reconstruction.

2 open findings

• Preserve array and data-table formulas deterministically · New
• Use print-area range objects for spaced and disjoint areas · New

4 resolved since last review

• Record and verify the build-time openpyxl version
• Validate user-defined column headers before workbook creation
• Remove image inspection from the workbook-only workflow
• Add explicit inspection and mapping steps to the workflow

🧠 Review effort: Balanced

[furukawa658]

Addressed the 2 follow-up review comments in 36bfe3e:

• Deterministically serialize array and data-table formulas from OOXML <f> elements and attributes while preserving kind: "formula" so object memory addresses never appear in the inventory.
• Read print-area ranges directly from sheet._print_area.ranges instead of reparsing the string representation with MultiCellRange, supporting sheet names with spaces and disjoint print areas.

/request-review

[Copilot]

🟡 Changes recommended

Merged-range expansion can bypass memory safeguards, and table-header validation and visibility assertions need correction.

1 open finding

• Bound merged-range expansion before loading workbooks · New

2 resolved since last review

• Use print-area range objects for spaced and disjoint areas
• Preserve array and data-table formulas deterministically

Previously missed (2)

In code that hasn't changed since last review

Enforce Excel's 255-character table column name limit

skills/​excel-design-reconstruction/​scripts/​reconstruct.py:402

Excel table column names are limited to 255 characters, but validate_text permits 32,767. A 256-character header therefore passes validation and becomes a table-column name that Excel cannot preserve as approved. Regenerating the same package with openpyxl does not detect this incompatibility. Reject headers longer than 255 characters here so fingerprinting and building enforce the table-specific limit.

Compare visibility flags explicitly against "True"

skills/​excel-design-reconstruction/​tests/​test_reconstruction.py:244

These inventory flags are strings, so both "True" and "False" satisfy assertTrue. The checks therefore pass even if hidden rows and columns are incorrectly reported as visible. Compare against "True" explicitly.

🧠 Review effort: Balanced

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[furukawa658]

Addressed the new merged-range expansion finding in a8b647c. The SAX preflight now computes the non-anchor cell allocation for every worksheet and enforces a cumulative workbook-wide limit before openpyxl loads the workbook. Regression tests cover the reported A1:Z1048576 range on an unselected worksheet and cumulative allocation across worksheets.

/request-review