Skip to content

fix(skills): close xlsx-official create.md library table row - #2624

Merged
yanyihan-xiaomi merged 1 commit into
mainfrom
fix/xlsx-create-md-table
Oct 9, 2026
Merged

yanyihan-xiaomi merged 1 commit into
mainfrom
fix/xlsx-create-md-table

Conversation

@yanyihan-xiaomi

Copy link
Copy Markdown
Collaborator

Summary

  • Fix the unclosed Library choice markdown table in packages/cli/src/skill/builtin/.bundle/xlsx-official/create.md: the last row had only 2 of 3 columns.
  • Fill Library with `pandas` + `openpyxl` and keep the original hybrid workflow text in Why.
  • Run Prettier on that file so the table columns align.

Verification

  • Fence-aware markdown table audit across the repo: issues=0 (the only other hit was a template row inside a code fence in deep-research/reference/report.md).
  • bun run --cwd packages/cli test test/skill → 92 pass / 0 fail.
  • prettier --check on the touched file passes.

Note

There is no dedicated markdown table lint script in the repo today. script/format.ts only runs Prettier (--write .), and CI lint is oxlint-only. Prettier currently reports style drift in ~148 markdown files repo-wide; that is formatting, not structural table breakage, and is left out of this PR.

@yanyihan-xiaomi
yanyihan-xiaomi merged commit f98dcd1 into main Oct 9, 2026
6 checks passed
@yanyihan-xiaomi
yanyihan-xiaomi deleted the fix/xlsx-create-md-table branch October 9, 2026 08:05
onlyfeng added a commit to onlyfeng/MiMo-Code that referenced this pull request Oct 10, 2026
Adopt upstream's opt-in Claude Code session import on startup (XiaomiMiMo#2621):
MIMOCODE_ENABLE_CLAUDE_IMPORT replaces MIMOCODE_DISABLE_CLAUDE_IMPORT,
startup import is off by default, and manual `session import-claude`
is unchanged. Bundled mimocode-docs config guidance documents the new
selector; the xlsx-official library-choice table row is closed (XiaomiMiMo#2624).

No fork adaptation: index.ts and xlsx-official/create.md match upstream
byte for byte; flag.ts and mimocode-docs/reference/config.md keep their
pre-merge fork drift unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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