This repository was archived by the owner on Aug 8, 2026. It is now read-only.
Port improvements from go-app-starter template - #516
Merged
Conversation
Use ESLint's flat configuration and HTML plugin to lint JavaScript in tracked ScreenJournal templates alongside tracked JavaScript files.
Initialize each SQLite connection with connection-scoped pragmas so pooled connections enforce foreign keys and Litestream settings consistently.
Run ScreenJournal through an explicit HTTP server handler and drain active requests before closing SQLite, allowing Litestream to perform its final sync within Fly's termination budget.
Keep production asset caching while serving no-store in dev builds so browser refreshes always retrieve rebuilt JavaScript and CSS.
mtlynch
force-pushed
the
template-updates-2026-08-08
branch
2 times, most recently
from
August 8, 2026 20:07
05fd699 to
7e8d44b
Compare
Give every Playwright test its own ScreenJournal server and SQLite database while preserving the test-scoped TMDB mock and poster stubs.
Validate Go import grouping through GCI, resolve development tools through GOBIN, reject unsupported test options, and reload JavaScript changes under modd.
Display Go VCS build information on ScreenJournal's about page when it is available, without inventing values for builds that lack source metadata. Add the build metadata package to the Docker build context.
Upgrade Playwright to 1.58.2, including CircleCI's matching browser image, and golang.org/x/sys to v0.41.0 while retaining ScreenJournal's newer Node and Flyctl pins.
Expose ScreenJournal build, test, lint, formatting, and E2E packages through Nix with minimal relevant inputs. Make the external Go test-package check enforce the project convention.
Replace vendored Bootstrap 5.2.2 assets with 5.3.8 and load the updated assets from ScreenJournal's base layout.
Port the complete applicable template guidance, retain ScreenJournal's PII logging rule, and use pnpm terminology for fixed-output dependencies.
mtlynch
force-pushed
the
template-updates-2026-08-08
branch
from
August 8, 2026 20:12
7e8d44b to
6cd604f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.