Merge main into next (closes #132) #133

Merged

clawbot merged 2 commits from issue-132-merge-main into next

2026-10-20 11:10:40 +02:00

Author SHA1 Message Date

Author	SHA1	Message	Date
clawbot	eea90b01e5	Merge main into next (closes #132 ) check / check (push) Waiting to run Details main held only the data race and test timeout fix, which next already has as `915f56e`, so every conflict is resolved to next's version and the merge changes no file. Model: opus-5-5	2026-10-20 08:42:53 +00:00
clawbot	e3324407ca	Fix the data races and the test timeout that fail make test (closes #104 ) check / check (push) Successful in 1m19s Details The HTTP server built its router inside the goroutine that starts serving, so the start hook returned before the router existed, and the internal/handlers tests raced with it or hit a nil router. The start hook now runs configure, enableSentry and SetupRoutes, in that order, then serves in the background. The goroutine that sends queued messages to an IRC client read c.nick without c.mu while NICK changed it. Every such read now takes the lock. Under -race in the Docker build, internal/handlers takes over 30s on database work, not clock waits, so both go test runs in make test use -timeout 120s. The \|\| retry stays (#101). Model: opus-5-5	2026-10-02 10:13:28 +02:00

clawbot

eea90b01e5

Merge main into next (closes #132 )

check / check (push) Waiting to run

Details

main held only the data race and test timeout fix, which next already
has as 915f56e, so every conflict is resolved to next's version and the
merge changes no file.

Model: opus-5-5

2026-10-20 08:42:53 +00:00

clawbot

e3324407ca

Fix the data races and the test timeout that fail make test (closes #104 )

check / check (push) Successful in 1m19s

Details

The HTTP server built its router inside the goroutine that starts
serving, so the start hook returned before the router existed, and
the internal/handlers tests raced with it or hit a nil router. The
start hook now runs configure, enableSentry and SetupRoutes, in that
order, then serves in the background.

The goroutine that sends queued messages to an IRC client read c.nick
without c.mu while NICK changed it. Every such read now takes the lock.

Under -race in the Docker build, internal/handlers takes over 30s on
database work, not clock waits, so both go test runs in make test use
-timeout 120s. The || retry stays
(#101).

Model: opus-5-5

2026-10-02 10:13:28 +02:00