[compscidr]

Main has been red since #687 renamed the module to github.com/goblogplatform/goblog. PRs branched earlier (#688–#693) merged with old goblog/... imports.

• Rewrite stale import lines to the module path. Left intentional "goblog/plugin" Yaegi alias strings in plugin/symbols.go, loader_test.go, validate_test.go untouched.
• blog/post_type_test.go and admin/crud_validation_test.go used a hand-rolled FuncMap missing commaList (from feat: optional person schema fields (jobTitle, worksFor, alumniOf, knowsAbout) #685); switched to theme.FuncMap() like other tests.

Verified locally: go build ./... and go test -race ./... pass.

🤖 Generated with Claude Code

[Copilot]

🟢 Approval recommended

The changes are mechanical module-path import rewrites plus a FuncMap switch that matches the established pattern at every other call site, and I verified completeness with no stale imports or compile issues remaining.

No open issues 🎉

What changed in this PR

This PR restores the main branch CI, which went red after #687 renamed the Go module to github.com/goblogplatform/goblog. PRs branched before that rename (#688–#693) merged with stale goblog/... import lines. The PR rewrites those stale imports to the new module path and, in two tests, replaces a hand-rolled template FuncMap (which was missing the commaList function introduced by #685) with the canonical theme.FuncMap().

I verified:

• The module path in go.mod is github.com/goblogplatform/goblog, matching the rewritten imports.
• theme.FuncMap() (theme/load.go:26-41) provides both rawHTML and commaList; commaList is referenced by templates/shared/_head.html:98, so the previous hand-rolled maps would have failed ParseGlob. The switch is correct and necessary.
• The two changed test files are external test packages (blog_test, admin_test), so adding the theme import introduces no import cycle.
• All remaining goblog/... occurrences are the intentional Yaegi alias strings (plugin/symbols.go, loader_test.go, validate_test.go, hello.go.example) left untouched as described, not real imports.
• Completeness: all other template-parsing test sites already use theme.FuncMap(), so no other test file remains broken.

Changes:

• Rewrite stale goblog/... import lines to the github.com/goblogplatform/goblog/... module path across several (mostly test) files.
• Switch blog/post_type_test.go and admin/crud_validation_test.go to theme.FuncMap() to pick up the commaList template function.

File	Description
wizard/​handlers_test.go	Rewrites datadir import to module path
tools/​seed_backlinks_test.go	Rewrites blog/tools imports to module path
plugin/​wasm/​contract.go	Rewrites plugin import to module path
plugin/​posts.go	Rewrites blog import to module path
plugin/​posts_test.go	Rewrites blog/plugin imports to module path
blog/​post_type_test.go	Module-path imports + switches to theme.FuncMap() for commaList
admin/​settings_api_test.go	Rewrites admin/auth/blog/plugin imports to module path
admin/​login_methods.go	Rewrites datadir import to module path
admin/​login_methods_test.go	Rewrites admin/blog/datadir imports to module path
admin/​crud_validation_test.go	Module-path imports + switches to theme.FuncMap() for commaList

_{Review effort: Balanced}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!