rubyforgood / rubyforgood/casa
Deploy log cleanup / hardening (casa-qa v3413)
Nobody has claimed this yet.
- Dominant language
- Ruby
- Stars
- 381
- Forks
- 542
- Avg merge
- 3d 15h
- Merged PRs (30d)
- 47
Description
Deploy log cleanup / hardening
The casa-qa Heroku build (v3413, Heroku-26 stack) succeeds and deploys, but the build log surfaces several warnings worth cleaning up. Full TODO plan gist: https://gist.github.com/compwron/56813f4c5a6665d6d284c7d50b3d8f07
Items inline below, roughly in priority order.
High priority (real impact)
- Disable on-demand asset compilation in production. Log warns
config.assets.compile = truein production. Assets are already precompiled at build time (rake assets:precompileruns fine), so on-demand compile only adds request latency and load. → Setconfig.assets.compile = falseinconfig/environments/production.rb. - Commit a normalized Gemfile.lock. Build warns
ffi-1.17.2platform-specific gem is installed but the lockfile only has the generic ruby version. → Runbundle lock --normalize-platforms(orbundle lock --add-platform x86_64-linux) and commit the updatedGemfile.lock.
Medium priority (hygiene)
- Address npm vulnerabilities.
4 vulnerabilities (3 moderate, 1 high)in the full tree;1 moderateremains in production deps after pruning. → Runnpm audit, thennpm audit fix. Note EOL/vulnerable build tooling:eslint@8.57.1,glob@10.5.0. - Bump Ruby patch version. Deploying on
ruby-4.0.3; Heroku recommends4.0.5(security + bug fixes). → Update.ruby-versionto4.0.5, test, redeploy. - Verify the release-phase command actually succeeded. A release command is declared (
this new release will not be available until the command succeeds); the log ends before showing its output. → Confirm the release/migration step passed in the release-phase logs.
Low priority (noise / optional)
- Refresh deprecated npm packages. Transitive deprecations from build/lint tooling:
rimraf@3,glob@7/glob@10.5.0,inflight@1.0.6(memory leak),coffee-script,lodash.template,whatwg-encoding@3.1.1. Runtime unaffected — candidate for a dependency refresh. - Review npm install-script allowlist.
esbuild@0.28.1andunrs-resolver@1.12.2have install scripts not yet onallowScripts. Build works; review/approve only if hardening the supply chain. - (Cosmetic) npm minor upgrade available 11.16.0 → 11.18.0.
Notes
- The one-time cache purge (
Changing stack from heroku-22 to heroku-26) explains this build's slowness. Subsequent builds should be cached again — no action needed.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with config/environments/production.rb, Gemfile.lock, and .ruby-version, then review the npm audit output and release-phase logs. Work through the checklist items that are applicable, including asset compilation, dependency and Ruby updates, and install-script review. Done means the selected changes are committed, tests and deployment checks pass, and the release or migration step is verified.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- node.js, rails, ruby
- Domain
- build-system, devops, release, security
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100