• Joined on 2026-02-08
clawbot opened issue sneak/upaas#133 2026-02-26 11:10:08 +01:00
Missing .editorconfig file
clawbot created pull request sneak/upaas#131 2026-02-26 11:09:48 +01:00
fix: simplify CI to docker build only (closes #130)
clawbot pushed to fix/ci-docker-build-only at sneak/upaas 2026-02-26 11:09:31 +01:00
55069d6845 fix: simplify CI to docker build only (closes #130)
clawbot created branch fix/ci-docker-build-only in sneak/upaas 2026-02-26 11:09:31 +01:00
clawbot commented on pull request sneak/upaas#126 2026-02-26 11:06:20 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Fixed — Docker doesnt support inline comments on FROM lines. Moved the # golang:1.25-alpine and # alpine:3.19 comments to their own line above each FROM statement.

make check passes.…

clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-26 11:06:14 +01:00
91d6da0796 fix: move inline comments above FROM lines (fixes docker build)
clawbot commented on pull request sneak/upaas#126 2026-02-26 11:02:49 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Rebased onto origin/main and resolved merge conflict (static/js/app.js was deleted on main during the JS module refactor — accepted the deletion since the PR's prettier formatting changes…

clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-26 11:02:43 +01:00
57e0735afa docs: expand Important note — HOST_DATA_DIR must be absolute path
2eeead7e64 docs: clarify UPAAS_DATA_DIR default is for local dev only
76fe014e9a docs: remove relative path default for HOST_DATA_DIR in docker-compose example
f36732eaf5 refactor: remove internal/domain package, move types to correct packages
3a1b1e3cd4 refactor: add String() methods to domain types, replace string() casts
Compare 11 commits »
clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-26 11:02:10 +01:00
f69cc5bb23 docs: expand Important note — HOST_DATA_DIR must be absolute path
3b740a88b0 docs: clarify UPAAS_DATA_DIR default is for local dev only
87a412379a docs: remove relative path default for HOST_DATA_DIR in docker-compose example
832f57a7f5 refactor: remove internal/domain package, move types to correct packages
ae3282218b refactor: add String() methods to domain types, replace string() casts
Compare 11 commits »
clawbot commented on pull request sneak/upaas#126 2026-02-26 11:01:56 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Rebased onto origin/main and resolved merge conflict (static/js/app.js was deleted on main during the JS module refactor — accepted the deletion since the PR's prettier formatting changes…

clawbot commented on pull request sneak/upaas#126 2026-02-26 11:01:55 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Rebased onto main and resolved the merge conflict (static/js/app.js was deleted on main when it was split into modules; the PR had cosmetic formatting changes to the old file, so the deletion…

clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-26 11:01:47 +01:00
17e9aba63c docs: expand Important note — HOST_DATA_DIR must be absolute path
364c2a7abe docs: clarify UPAAS_DATA_DIR default is for local dev only
3714ee50ce docs: remove relative path default for HOST_DATA_DIR in docker-compose example
ec436f4bf7 refactor: remove internal/domain package, move types to correct packages
38895a1a39 refactor: add String() methods to domain types, replace string() casts
Compare 13 commits »
clawbot commented on pull request sneak/upaas#126 2026-02-24 00:45:04 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Code Review — Merge-Ready

Rebased on main (already up-to-date, no conflicts). make check passes cleanly.

==> All checks passed!

All tests pass, linter clean, build succeeds.…

clawbot commented on pull request sneak/upaas#129 2026-02-24 00:44:18 +01:00
Refactor: break up app.js into smaller modules

make check output

All tests passed:
- git.eeqj.de/sneak/upaas/internal/models: PASS (3.773s)
- git.eeqj.de/sneak/upaas/internal/service/app: PASS (2.880s)
- git.eeqj.de/sneak/upaas/intern…
clawbot commented on pull request sneak/upaas#126 2026-02-23 22:17:33 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Pushed 5791d33: expanded the Important note to explain that HOST_DATA_DIR must be an absolute host path, why relative paths break, and added a usage example.

All previous commits already…

clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-23 22:17:24 +01:00
5791d33720 docs: expand Important note — HOST_DATA_DIR must be absolute path
clawbot commented on pull request sneak/upaas#126 2026-02-23 22:16:17 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Fixed the remaining relative path default in the README config table:

  1. HOST_DATA_DIR in docker-compose — already fixed in 4d53ee5 (no more :-./data fallback)
  2. UPAAS_DATA_DIR config…
clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-23 22:16:09 +01:00
e115aa765a docs: clarify UPAAS_DATA_DIR default is for local dev only
clawbot commented on pull request sneak/upaas#126 2026-02-23 22:15:18 +01:00
Fix 1.0 audit bugs (closes #120, closes #121, closes #122, closes #123, closes #124, closes #125)

Removed relative path default (:-./data) for HOST_DATA_DIR in the README docker-compose example — both the volume mount and env var now require it to be explicitly set. Also updated the…

clawbot pushed to fix/audit-bugs-120-125 at sneak/upaas 2026-02-23 22:15:14 +01:00
4d53ee5b06 docs: remove relative path default for HOST_DATA_DIR in docker-compose example