clawbot
0 Followers · 0 Following
  • Joined on 2026-02-08
Repositories Projects Packages Code Public Activity Starred Repositories
clawbot commented on pull request sneak/AutistMask#145 2026-03-18 04:30:31 +01:00
feat: add About well to settings with build info and debug easter egg

Rework Summary

Fix: Removed dead DEBUG import from src/popup/views/helpers.js (line 3).

The DEBUG constant from ../../shared/constants was imported but never used — `updateDebugBa…

clawbot pushed to feat/issue-144-settings-about at sneak/AutistMask 2026-03-18 04:30:13 +01:00
5ad9e9d09d fix: remove dead DEBUG import from helpers.js
clawbot commented on pull request sneak/neoirc#82 2026-03-18 04:28:57 +01:00
feat: add username/hostname support with IRC hostmask format

Review: PR #82 — Username/Hostname Support (post-rework #7 — build/test fixes)

clawbot commented on pull request sneak/pixa#36 2026-03-18 04:28:18 +01:00
Move schema_migrations table creation into 000.sql

Review: PR #36 — Move schema_migrations table creation into 000.sql (post-rework v3)

Closes issue #29.

-…

clawbot commented on pull request sneak/neoirc#82 2026-03-18 04:24:12 +01:00
feat: add username/hostname support with IRC hostmask format

Rework: Fix test speed and output verbosity

Addressed sneak's feedback on test performance and output size.

Changes

Makefile — removed -v flag from test target. Output now shows…

clawbot pushed to feature/username-hostname-support at sneak/neoirc 2026-03-18 04:23:57 +01:00
3e6dc108db fix: speed up tests and reduce output verbosity
clawbot commented on pull request sneak/pixa#36 2026-03-18 04:23:05 +01:00
Move schema_migrations table creation into 000.sql

Rework: Moved bootstrap INSERT into 000.sql

Per sneak's review — the SQL file should be self-contained.

Changes

internal/database/schema/000.sql — added `INSERT OR IGNORE INTO…

clawbot pushed to feature/migrations-table-schema-file at sneak/pixa 2026-03-18 04:22:55 +01:00
0c8de65be8 Move bootstrap migration INSERT into 000.sql
clawbot commented on pull request sneak/pixa#37 2026-03-18 04:17:27 +01:00
Bound imageprocessor.Process input read to prevent unbounded memory use

Good question — looking at the package, internal/imgcache/ currently contains the image processor, HTTP fetcher, host whitelist, service orchestrator, and server/handlers all in one package.…

clawbot commented on pull request sneak/pixa#37 2026-03-18 04:00:01 +01:00
Bound imageprocessor.Process input read to prevent unbounded memory use

Code Review: PR #37 — Post-Rework Review

clawbot pushed to style/constructor-naming-params at sneak/prompts 2026-03-18 03:57:31 +01:00
f05fdf6674 style: Params struct required even for single arguments
clawbot commented on pull request sneak/pixa#37 2026-03-18 03:55:30 +01:00
Bound imageprocessor.Process input read to prevent unbounded memory use

Rework: Constructor refactored to params pattern

Addressed review feedback — refactored the image processor constructor from a…

clawbot pushed to fix/bounded-processor-read at sneak/pixa 2026-03-18 03:55:17 +01:00
d36e511032 refactor: use Params struct for imageprocessor constructor
18f218e039 bound imageprocessor.Process input read to prevent unbounded memory use
9c29cb57df feat: parse version prefix from migration filenames (#33)
Compare 3 commits »
clawbot commented on pull request sneak/pixa#36 2026-03-18 03:54:08 +01:00
Move schema_migrations table creation into 000.sql

Review: PR #36 — Move schema_migrations table creation into 000.sql (post-rework)

clawbot created pull request sneak/prompts#19 2026-03-18 03:54:00 +01:00
style: strengthen constructor naming and Params struct rules
clawbot pushed to style/constructor-naming-params at sneak/prompts 2026-03-18 03:53:47 +01:00
2b674c7d22 style: strengthen constructor naming and Params struct rules
clawbot created branch style/constructor-naming-params in sneak/prompts 2026-03-18 03:53:47 +01:00
clawbot commented on pull request sneak/pixa#36 2026-03-18 03:49:41 +01:00
Move schema_migrations table creation into 000.sql

Rework: Removed all backwards compatibility code

Per sneak's review — pixa is pre-1.0 with no installed base, so all legacy DB handling was unnecessary complexity.

Changes

**`internal/da…

clawbot pushed to feature/migrations-table-schema-file at sneak/pixa 2026-03-18 03:49:20 +01:00
8b6aa74951 Remove backwards compat code: no installed base pre-1.0
clawbot commented on pull request sneak/neoirc#82 2026-03-18 03:47:27 +01:00
feat: add username/hostname support with IRC hostmask format

Review: PR #82 — Username/Hostname Support (post-rebase #6)