Skip to content

fix: resolve acceptance test findings#9

Merged
K1-R1 merged 6 commits intomainfrom
fix/acceptance-test-findings
Mar 17, 2026
Merged

fix: resolve acceptance test findings#9
K1-R1 merged 6 commits intomainfrom
fix/acceptance-test-findings

Conversation

@K1-R1
Copy link
Owner

@K1-R1 K1-R1 commented Mar 17, 2026

Summary

  • Fix .gitignore crash on read-only repos with external --output-dir
  • Add symlink .gitignore guard
  • Fix version-pinned curl URLs (v1.0.0 → main)
  • Add SHA256 checksum to README manual install
  • Use $RUNNER_TEMP in release workflow
  • Add sudo warning to install.sh
  • README: TOC, uninstall section, agent/CI docs, colour env vars, install.sh env vars
  • Agent example in --help output
  • 3 new bats tests
  • Bump to v1.0.2

Test plan

  • shellcheck clean
  • shfmt clean
  • 231/231 bats tests pass

K1-R1 added 6 commits March 17, 2026 01:26
- Skip .gitignore modification when --output-dir is outside the repo root
  (fixes crash on read-only repos used by sandboxed agents)
- Skip .gitignore modification when .gitignore is a symlink (prevents
  overwriting symlink target)
- Add agent/CI example to --help: smoosh --no-interactive --json .
- Add 3 new bats tests covering all changes

Signed-off-by: K1-R1 <77465250+K1-R1@users.noreply.github.com>
…TEMP

- Change README and install.sh curl URLs from v1.0.0 to main branch
  (install.sh resolves latest version at runtime; pinning the URL to a
  tag caused stale installs)
- Add SHA256 checksum verification to README manual install method
- Document install.sh env vars (SMOOSH_VERSION, SMOOSH_NO_CONFIRM, etc.)
- Add sudo elevation warning before fallback in install.sh
- Use $RUNNER_TEMP instead of /tmp in release.yml
- Fix shfmt formatting in install.sh

Signed-off-by: K1-R1 <77465250+K1-R1@users.noreply.github.com>
- Add navigation table of contents at the top
- Add uninstall section covering Homebrew and manual methods, with
  shadow binary warning
- Add "Agents and CI pipelines" section with recommended flags,
  JSON schema example, and pre-flight pattern
- Add --no-color to configuration reference table
- Document colour env var precedence (NO_COLOR, FORCE_COLOR, CLICOLOR)

Signed-off-by: K1-R1 <77465250+K1-R1@users.noreply.github.com>
Signed-off-by: K1-R1 <77465250+K1-R1@users.noreply.github.com>
Signed-off-by: K1-R1 <77465250+K1-R1@users.noreply.github.com>
The include-hidden-md golden file expected a .gitignore entry that was
only created because of the B1 bug (ensure_output_dir fell through to
modify .gitignore when --output-dir was external). With the fix, the
.gitignore is correctly left untouched, so the golden file no longer
includes it.

Signed-off-by: K1-R1 <77465250+K1-R1@users.noreply.github.com>
@K1-R1 K1-R1 merged commit 79733a7 into main Mar 17, 2026
6 checks passed
@K1-R1 K1-R1 deleted the fix/acceptance-test-findings branch March 17, 2026 02:07
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.

1 participant