fix(pack): ignore .oo-thumbnail.json

[BlackHole1]

No description provided.

[Copilot]

Pull Request Overview

This PR aims to address packaging behavior for .oo-thumbnail.json as indicated in the title. Key changes include test fixture additions, updates to package file generation in npm.ts with corresponding tests, and an additional test in the pack command to validate the packaged files.

Reviewed Changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/fixtures/pack_ignore/src/index.ts	Adds a basic fixture file.
tests/fixtures/pack_ignore/package.oo.yaml	Introduces a test package fixture.
src/utils/npm.ts	Updates package files list to include .oo-thumbnail.json.
src/utils/npm.test.ts	Adjusts tests to reflect the inclusion of .oo-thumbnail.json.
src/cmd/pack.test.ts	Adds a test to validate that the tar archive includes .oo-thumbnail.json.

Files not reviewed (2)

• tests/fixtures/pack_ignore/.gitignore: Language not supported
• tests/fixtures/pack_ignore/package.json: Language not supported

Comments suppressed due to low confidence (1)

src/cmd/pack.test.ts:104

• This test expects the tar archive to contain 'package/package/.oo-thumbnail.json', which appears to duplicate the 'package' folder structure and deviates from the intended file exclusion suggested by the PR title. Verify if this double nesting is correct or if the file should be omitted.

            expect(result).toContainEqual("package/package/.oo-thumbnail.json");

[Copilot]

Pull Request Overview

This PR updates the packaging functionality to handle the .oo-thumbnail.json file correctly during packaging.

• Updates the package.json generation logic to include "package/.oo-thumbnail.json".
• Enhances test coverage to explicitly verify that .oo-thumbnail.json is present in the package tarball.
• Adds a new test fixture and package configuration for the pack_ignore scenario.

Reviewed Changes

Copilot reviewed 5 out of 9 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tests/fixtures/pack_ignore/src/index.ts	Added a simple fixture comment ("// PASS") for the pack_ignore tests
tests/fixtures/pack_ignore/package.oo.yaml	Introduces a minimal package configuration for testing the pack_ignore setup
src/utils/npm.ts	Updates package file list to include the .oo-thumbnail.json file
src/utils/npm.test.ts	Updates tests to verify that .oo-thumbnail.json is included during packaging
src/cmd/pack.test.ts	Adds a new test case ensuring that .oo-thumbnail.json is not ignored

Files not reviewed (4)

• tests/fixtures/pack_ignore/.oo-thumbnail.json: Language not supported
• tests/fixtures/pack_ignore/_gitignore: Language not supported
• tests/fixtures/pack_ignore/foo.txt: Language not supported
• tests/fixtures/pack_ignore/package.json: Language not supported

Comments suppressed due to low confidence (1)

src/cmd/pack.test.ts:86

• The test title indicates that .oo-thumbnail.json should be included, but the PR title suggests it should be ignored. Consider aligning the test description with the PR title for clarity.

it("should not ignore .oo-thumbnail.json", async (ctx) => {