Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Oct 13, 2025

  • Update tsconfig.json to output to distribution/ folder
  • Update package.json exports and files to reference distribution/
  • Update .gitignore to exclude distribution/ folder instead of individual build files
  • Update test imports that reference ../index.js to use ../distribution/index.js
  • Run build to generate files in new location
  • Run tests to verify everything works (all 34 tests pass)
  • Verified the module can be imported successfully from distribution/
  • Remove trailing comma from tsconfig.json compilerOptions

Summary: Successfully moved all build outputs from the root directory to the distribution/ folder. Source files were already in the source/ directory. All tests pass and the module imports correctly.

Original prompt

Move all source files in the repository into a folder named 'source' and ensure that output files are directed into a folder named 'distribution'. Update any configurations, imports, or scripts in the repository to reflect these changes. Test the changes to confirm that the project functions correctly with the new folder structure.

Mentioned in fregante/chrome-webstore-upload-cli#96 (comment)

This pull request was created as a result of the following prompt from Copilot chat.

Move all source files in the repository into a folder named 'source' and ensure that output files are directed into a folder named 'distribution'. Update any configurations, imports, or scripts in the repository to reflect these changes. Test the changes to confirm that the project functions correctly with the new folder structure.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@Copilot Copilot AI changed the title [WIP] Move source files to 'source' folder and update configurations Move build outputs to distribution folder Oct 13, 2025
@Copilot Copilot AI requested a review from fregante October 13, 2025 03:31
Copilot finished work on behalf of fregante October 13, 2025 03:31
tsconfig.json Outdated
"extends": "@sindresorhus/tsconfig",
"compilerOptions": {
"outDir": ".",
"outDir": "distribution",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the default in the extended config, it can be deleted

@copilot

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the trailing comma in commit e2a84ab.

@fregante fregante marked this pull request as ready for review October 13, 2025 03:33
Copy link
Contributor Author

Copilot AI commented Oct 13, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • www.googleapis.com

If you need me to access, download, or install something from one of these locations, you can either:

@Copilot Copilot AI requested a review from fregante October 13, 2025 03:37
Copilot finished work on behalf of fregante October 13, 2025 03:37
@fregante fregante merged commit 296079a into main Oct 13, 2025
6 checks passed
@fregante fregante deleted the copilot/move-source-files-structure branch October 13, 2025 05:17
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.

2 participants