clawbot
0 Followers · 0 Following
  • Joined on 2026-02-08
Repositories Projects Packages Code Public Activity Starred Repositories
clawbot commented on pull request sneak/pixa#42 2026-03-26 01:50:21 +01:00
refactor: extract magic byte detection into internal/magic package

Review: PASS ✅

Clean extraction of magic byte detection into internal/magic/ package.

Checklist

clawbot created pull request sneak/pixa#42 2026-03-26 01:47:00 +01:00
refactor: extract magic byte detection into internal/magic package
clawbot pushed to refactor/extract-magic-package at sneak/pixa 2026-03-26 01:46:46 +01:00
dedc729f37 refactor: extract magic byte detection into internal/magic package
clawbot created branch refactor/extract-magic-package in sneak/pixa 2026-03-26 01:46:46 +01:00
clawbot commented on pull request sneak/neoirc#95 2026-03-25 22:47:06 +01:00
feat: move schema_migrations into 000_bootstrap.sql

Re-Review: PR #95 — Move schema_migrations into 000.sql (post-rework)

Result: PASS ✅

Requirements Checklist

clawbot commented on pull request sneak/neoirc#95 2026-03-25 22:46:31 +01:00
feat: move schema_migrations into 000_bootstrap.sql

Fixed: renamed 000_bootstrap.sql → 000.sql and updated the reference in db.go. docker build --no-cache . passes.

clawbot pushed to feature/schema-migrations-000sql at sneak/neoirc 2026-03-25 22:46:26 +01:00
f6d33e3ceb fix: rename 000_bootstrap.sql to 000.sql per issue spec
clawbot pushed to feature/schema-migrations-000sql at sneak/neoirc 2026-03-25 22:46:15 +01:00
106a8e1a84 fix: rename 000_bootstrap.sql to 000.sql per issue spec
clawbot commented on pull request sneak/neoirc#95 2026-03-25 22:44:52 +01:00
feat: move schema_migrations into 000_bootstrap.sql

Review: PR #95 — Move schema_migrations into 000.sql

Result: FAIL — 1 blocking issue

Requirements Checklist

clawbot created pull request sneak/neoirc#95 2026-03-25 22:42:09 +01:00
feat: move schema_migrations into 000_bootstrap.sql
clawbot pushed to feature/schema-migrations-000sql at sneak/neoirc 2026-03-25 22:41:52 +01:00
a883d708a2 feat: move schema_migrations table creation into 000_bootstrap.sql
clawbot created branch feature/schema-migrations-000sql in sneak/neoirc 2026-03-25 22:41:52 +01:00
clawbot pushed to feature/irc-protocol-listener at sneak/neoirc 2026-03-25 22:22:51 +01:00
ac89a99c35 refactor: migrate HTTP handlers to shared service layer
clawbot commented on pull request sneak/neoirc#94 2026-03-25 21:51:54 +01:00
feat: add traditional IRC wire protocol listener (closes #89)

Review: PR #94 — IRC Wire Protocol Listener (post-rework)

Result: FAIL — 3 blocking issues

Blocking Issues

  1. README inaccuracy: README says IRC listener is disabled "when…
clawbot commented on pull request sneak/neoirc#94 2026-03-25 21:47:19 +01:00
feat: add traditional IRC wire protocol listener (closes #89)

Review: PR #94 — IRC Wire Protocol Listener

Build Result

✅ docker build . passes cleanly. All tests green, 0 lint issues.

Cheating Check

✅ No changes to .golangci.yml,…

clawbot commented on pull request sneak/neoirc#94 2026-03-25 21:46:51 +01:00
feat: add traditional IRC wire protocol listener (closes #89)

Review: FAIL

docker build . passes clean (0 lint issues, all tests green). The IRC wire protocol implementation is solid work — good parser, comprehensive integration tests, clean Fx…

clawbot commented on pull request sneak/neoirc#94 2026-03-25 21:41:59 +01:00
feat: add traditional IRC wire protocol listener (closes #89)

Rework complete. All three review items addressed:

  1. Default IRC_LISTEN_ADDR to :6667 — config.go and README updated
  2. Removed nolint:funlen directives — functions broken into…
clawbot commented on pull request sneak/neoirc#94 2026-03-25 21:40:07 +01:00
feat: add traditional IRC wire protocol listener (closes #89)

Reworked per feedback:

  1. IRC_LISTEN_ADDR default changed to :6667 — listener is now enabled by default
  2. Removed all nolint:funlen directives — refactored long functions into…
clawbot pushed to feature/irc-protocol-listener at sneak/neoirc 2026-03-25 21:39:44 +01:00
2853dc8a1f refactor: shared service layer, default IRC port, smaller functions
clawbot commented on pull request sneak/neoirc#94 2026-03-25 21:19:22 +01:00
feat: add traditional IRC wire protocol listener (closes #89)

Fixed: changed IRC_LISTEN_ADDR default from "" to ":6667" in internal/config/config.go, and updated the README environment variable table to show ":6667" as the default.

make fmt,…