Skip to content

BrewFile support and splitting software into prompts/files#122

Merged
atomantic merged 3 commits intomainfrom
software_split
Jul 11, 2025
Merged

BrewFile support and splitting software into prompts/files#122
atomantic merged 3 commits intomainfrom
software_split

Conversation

@atomantic
Copy link
Copy Markdown
Owner

@atomantic atomantic commented Jul 11, 2025

addresses #39

Copilot AI review requested due to automatic review settings July 11, 2025 21:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the software installation configuration by splitting a single config into per-manager files, adds support for a project-level .BrewFile, and updates scripts and docs to load and prompt on the new files.

  • Split config.js into software/*.js files for each package manager
  • Introduce .BrewFile support for additional Homebrew packages
  • Update index.js, install.sh, and documentation to load software configs dynamically

Reviewed Changes

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

Show a summary per file
File Description
software/brew.js New split file listing Homebrew CLI utilities
software/cask.js New split file listing Homebrew GUI apps
software/npm.js New split file listing global NPM packages
software/mas.js New split file stub for Mac App Store apps
software/gem.js New split file stub for Ruby gems
software/README.md Doc added for each split config file
install.sh Updated prompt message to reference software configuration files
index.js Major refactor: dynamic imports of software/*.js, .BrewFile merge
homedir/.BrewFile Sample .BrewFile for additional Homebrew packages
README.md Updated top-level docs to reference the new software/ directory
Comments suppressed due to low confidence (2)

index.js:39

  • New .BrewFile parsing and dynamic imports deserve dedicated tests to validate that additional packages are merged correctly and prompts behave as expected.
  const brewFileExists = fs.existsSync("./homedir/.BrewFile");

index.js:67

  • [nitpick] The default property is not aligned under message, making the object literal formatting inconsistent. Re-indent for clarity and consistency.
        default: false,

Comment thread index.js Outdated
Comment thread index.js
atomantic and others added 2 commits July 11, 2025 14:37
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@atomantic atomantic self-assigned this Jul 11, 2025
Copy link
Copy Markdown
Owner Author

@atomantic atomantic left a comment

Choose a reason for hiding this comment

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

lgtm

@atomantic atomantic merged commit 43b869d into main Jul 11, 2025
1 check passed
@atomantic atomantic deleted the software_split branch February 4, 2026 23:08
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