Fix theme pull, push, and package to cover all theme files

[karreiro]

WHY are these changes introduced?

Shopify CLI does not handle DESIGN.md returned by the checksums API.

WHAT is this pull request doing?

Include DESIGN.md in theme pull, push, and package.

How to manually test your changes?

shopify theme pull
shopify theme push
shopify theme package

Notice that DESIGN.md is pulled, pushed, and included in the generated package.

Checklist

• I've considered possible cross-platform impacts (Mac, Linux, Windows)
• I've considered possible documentation changes
• I've considered analytics changes to measure impact
• The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

[Copilot]

🟡 Changes recommended

Correct the fixture checksum in the filesystem test and add the missing patch changeset.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds DESIGN.md support to theme pull, push, and package workflows.

Changes:

• Discovers and partitions DESIGN.md.
• Includes it in upload, deletion, and generated packages.
• Adds fixtures and test coverage.

File summaries

File	Summary
packages/theme/src/cli/utilities/theme-uploader.ts	Handles documentation ordering during synchronization.
packages/theme/src/cli/utilities/theme-uploader.test.ts	Tests DESIGN.md upload and deletion behavior.
packages/theme/src/cli/utilities/theme-fs.ts	Discovers and categorizes DESIGN.md.
packages/theme/src/cli/utilities/theme-fs.test.ts	Tests discovery and partitioning; expected checksum requires correction.
packages/theme/src/cli/utilities/fixtures/theme/DESIGN.md	Adds a DESIGN.md fixture.
packages/theme/src/cli/services/package.ts	Includes DESIGN.md in generated packages.
packages/theme/src/cli/services/package.test.ts	Verifies packaged DESIGN.md inclusion.

Review details

Suppressed comments (1)

packages/theme/src/cli/services/package.ts:10

• This adds user-facing theme packaging behavior, but there is no new .changeset file in the PR even though the checklist says one was added. Add a patch changeset for the theme package before merging.

  'DESIGN.md',

• Files reviewed: 7/7 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.