Skip to content

Setup fix#130

Merged
jugglingdev merged 7 commits into
Open-SGF:mainfrom
jugglingdev:setup-fix
Mar 15, 2026
Merged

Setup fix#130
jugglingdev merged 7 commits into
Open-SGF:mainfrom
jugglingdev:setup-fix

Conversation

@jugglingdev
Copy link
Copy Markdown
Contributor

@jugglingdev jugglingdev commented Mar 11, 2026

Added a composer setup script to clear config and cache to prevent caching issues when creating a new branch.

Made the Neon API variables nullable and created a private function to check that they are configured, which is called before the main fetch() function. Previously, there was a runtime error if those variables weren't configured which was preventing the composer config:clear and composer cache:clear commands from being run. Making them nullable stopped the runtime error and calling the private function ensures they are configured before making the API calls.

Also updated composer and npm.

@ThingEngineer
Copy link
Copy Markdown
Contributor

Looks good! There were still a few issues though when starting up a fresh clone and since I cannot push commits to this PR so I created a follow up to this in PR #131 for these issues:

  • make composer setup host-safe by avoiding DB-backed cache clearing during first-run setup making composer setup more idempotent across onboarding states (before and after Sail/DB availability)
  • clarify the first-run onboarding order in the README
  • align Sail and Composer on PHP 8.4 so containerized Composer and Artisan commands stop failing
  • stop tracking generated bootstrap cache files that appear to have been committed before .gitignore was already excluding that folder

@jugglingdev jugglingdev merged commit f7afb1d into Open-SGF:main Mar 15, 2026
3 checks passed
@jugglingdev jugglingdev deleted the setup-fix branch March 15, 2026 18:43
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.

5 participants