Add plt print command for purple playlists by statik · Pull Request #886 · kindlyops/vbs

statik · 2026-06-11T17:20:08Z

Summary

First phase of the plt command group for working with purple playlist
exports (a ZIP container holding a SQLite database, produced by the source
app for queuing media in live events).

This phase delivers vbs plt print <playlist>, which parses a playlist and
prints its cues offline — no media is downloaded.

Format sniffer detects playlists by content, not file extension, and
rejects invalid files with a named reason (not a zip, missing manifest,
non-SQLite database, or missing required tables). Unverified schema
versions warn but proceed when the required tables are present.
Parser reads the verified table contract into an ordered domain model:
items, published-media locations (pub/track, book/chapter, docid shapes), embedded
image cues, and segment markers.
plt print renders an aligned table (position, label, source, duration,
marker count, raw end action) with a --json flag for the same data.
Synthetic fixture generator builds playlist zips in-test (SQLite +
manifest + 1×1 JPEGs) covering every source shape, so tests never depend on
real exports.
Vendor-neutrality guard (scripts/check-vendor-neutral.sh, wired into
the lint workflow) rejects the publisher's identifiers in authored files.

modernc.org/sqlite (pure-Go, no CGO) is promoted from an indirect to a
direct dependency.

Test Plan

go test ./cmd/ green (helpers, sniffer, parser, print; tick
conversion, slugging, language resolution, all four source shapes)
bazel build //cmd:go_default_library and bazel test //cmd:go_default_test green
gofmt, go vet ./cmd/, go build ./..., go mod verify clean
funlen / gocyclo / lll within limits
Manually verified against a real 15-item export: 15 items, 10 markers
across 8 items, 1 image cue, correct source descriptions
Vendor guard passes on the tree and fails on an injected string

statik · 2026-06-11T17:21:56Z

+	"path/filepath"
+	"testing"
+
+	_ "modernc.org/sqlite"


🚫 [golangci] _{reported by reviewdog 🐶}
import 'modernc.org/sqlite' is not allowed from list 'Main' (depguard)

statik · 2026-06-11T17:21:57Z

+	"path/filepath"
+	"strings"
+
+	_ "modernc.org/sqlite" // registers the pure-Go "sqlite" database/sql driver


🚫 [golangci] _{reported by reviewdog 🐶}
import 'modernc.org/sqlite' is not allowed from list 'Main' (depguard)

statik · 2026-06-11T17:21:57Z

+		`INSERT INTO TagMap VALUES (3, 1, 3, 2)`,
+		`INSERT INTO TagMap VALUES (4, 1, 4, 3)`,
+
+		`INSERT INTO Location VALUES (1, NULL, NULL, 1102016935, 135, 'sjj', 420, 0)`,


🚫 [golangci] _{reported by reviewdog 🐶}
Duplicate words (NULL,) found (dupword)

statik · 2026-06-11T17:21:57Z

+		`INSERT INTO TagMap VALUES (4, 1, 4, 3)`,
+
+		`INSERT INTO Location VALUES (1, NULL, NULL, 1102016935, 135, 'sjj', 420, 0)`,
+		`INSERT INTO Location VALUES (2, 23, 5, NULL, NULL, 'nwt', 420, 0)`,


🚫 [golangci] _{reported by reviewdog 🐶}
Duplicate words (NULL,) found (dupword)

statik · 2026-06-11T17:21:57Z

+
+		`INSERT INTO Location VALUES (1, NULL, NULL, 1102016935, 135, 'sjj', 420, 0)`,
+		`INSERT INTO Location VALUES (2, 23, 5, NULL, NULL, 'nwt', 420, 0)`,
+		`INSERT INTO Location VALUES (4, NULL, NULL, 1112024040, 1, NULL, 420, 3)`,


🚫 [golangci] _{reported by reviewdog 🐶}
Duplicate words (NULL,) found (dupword)

statik · 2026-06-11T17:21:58Z

+}
+
+func TestSniffPlaylist_UnverifiedSchemaProceeds(t *testing.T) {
+	path := writePlaylistFixture(t, fixtureOptions{schemaVersion: 99})


🚫 [golangci] _{reported by reviewdog 🐶}
cmd.fixtureOptions is missing fields omitTables, omitManifest, notZip, corruptDB, databaseName (exhaustruct)

statik · 2026-06-11T17:21:58Z

+// fixtureSchema returns CREATE statements for every table except those omitted.
+func fixtureSchema(omit map[string]bool) []string {
+	tables := map[string]string{
+		"Tag":                     `CREATE TABLE Tag (TagId INTEGER PRIMARY KEY, Name TEXT, Type INTEGER)`,


🚫 [golangci] _{reported by reviewdog 🐶}
string Tag has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T17:21:58Z

+func fixtureSchema(omit map[string]bool) []string {
+	tables := map[string]string{
+		"Tag":                     `CREATE TABLE Tag (TagId INTEGER PRIMARY KEY, Name TEXT, Type INTEGER)`,
+		"TagMap":                  `CREATE TABLE TagMap (TagMapId INTEGER PRIMARY KEY, TagId INTEGER, PlaylistItemId INTEGER, Position INTEGER)`,


🚫 [golangci] _{reported by reviewdog 🐶}
string TagMap has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T17:21:58Z

+	tables := map[string]string{
+		"Tag":                     `CREATE TABLE Tag (TagId INTEGER PRIMARY KEY, Name TEXT, Type INTEGER)`,
+		"TagMap":                  `CREATE TABLE TagMap (TagMapId INTEGER PRIMARY KEY, TagId INTEGER, PlaylistItemId INTEGER, Position INTEGER)`,
+		"PlaylistItem":            `CREATE TABLE PlaylistItem (PlaylistItemId INTEGER PRIMARY KEY, Label TEXT, StartTrimOffsetTicks INTEGER, EndTrimOffsetTicks INTEGER, EndAction INTEGER, ThumbnailFilePath TEXT)`,


🚫 [golangci] _{reported by reviewdog 🐶}
string PlaylistItem has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T17:21:58Z

+		"Tag":                     `CREATE TABLE Tag (TagId INTEGER PRIMARY KEY, Name TEXT, Type INTEGER)`,
+		"TagMap":                  `CREATE TABLE TagMap (TagMapId INTEGER PRIMARY KEY, TagId INTEGER, PlaylistItemId INTEGER, Position INTEGER)`,
+		"PlaylistItem":            `CREATE TABLE PlaylistItem (PlaylistItemId INTEGER PRIMARY KEY, Label TEXT, StartTrimOffsetTicks INTEGER, EndTrimOffsetTicks INTEGER, EndAction INTEGER, ThumbnailFilePath TEXT)`,
+		"PlaylistItemLocationMap": `CREATE TABLE PlaylistItemLocationMap (PlaylistItemId INTEGER, LocationId INTEGER, MajorMultimediaType INTEGER, BaseDurationTicks INTEGER)`,


🚫 [golangci] _{reported by reviewdog 🐶}
string PlaylistItemLocationMap has 3 occurrences, make it a constant (goconst)

statik

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci

🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestResolveLanguage missing the call to method parallel in test Run (paralleltest)

vbs/cmd/plt_helpers_test.go

Line 59 in fbf96f6

for _, tc := range cases {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSlugify missing the call to method parallel (paralleltest)

vbs/cmd/plt_helpers_test.go

Line 78 in fbf96f6

func TestSlugify(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestSlugify missing the call to method parallel in test Run (paralleltest)

vbs/cmd/plt_helpers_test.go

Line 93 in fbf96f6

for _, tc := range cases {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestUniqueSlug missing the call to method parallel (paralleltest)

vbs/cmd/plt_helpers_test.go

Line 103 in fbf96f6

func TestUniqueSlug(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestParsePlaylist_NameAndOrder missing the call to method parallel (paralleltest)

vbs/cmd/plt_parse_test.go

Line 36 in fbf96f6

func TestParsePlaylist_NameAndOrder(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestParsePlaylist_PubTrackItem missing the call to method parallel (paralleltest)

vbs/cmd/plt_parse_test.go

Line 52 in fbf96f6

func TestParsePlaylist_PubTrackItem(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestParsePlaylist_BibleItemMarkers missing the call to method parallel (paralleltest)

vbs/cmd/plt_parse_test.go

Line 72 in fbf96f6

func TestParsePlaylist_BibleItemMarkers(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestParsePlaylist_ImageCue missing the call to method parallel (paralleltest)

vbs/cmd/plt_parse_test.go

Line 92 in fbf96f6

func TestParsePlaylist_ImageCue(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestParsePlaylist_DocidItem missing the call to method parallel (paralleltest)

vbs/cmd/plt_parse_test.go

Line 109 in fbf96f6

func TestParsePlaylist_DocidItem(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestDescribeSource missing the call to method parallel (paralleltest)

vbs/cmd/plt_print_test.go

Line 52 in fbf96f6

func TestDescribeSource(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestDescribeSource missing the call to method parallel in test Run (paralleltest)

vbs/cmd/plt_print_test.go

Line 80 in fbf96f6

for _, tc := range cases {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestBuildPrintView missing the call to method parallel (paralleltest)

vbs/cmd/plt_print_test.go

Line 89 in fbf96f6

func TestBuildPrintView(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestRenderJSON_RoundTrips missing the call to method parallel (paralleltest)

vbs/cmd/plt_print_test.go

Line 122 in fbf96f6

func TestRenderJSON_RoundTrips(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestRenderText_ContainsKeyData missing the call to method parallel (paralleltest)

vbs/cmd/plt_print_test.go

Line 139 in fbf96f6

func TestRenderText_ContainsKeyData(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_AcceptsValidRenamedFile missing the call to method parallel (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 22 in fbf96f6

func TestSniffPlaylist_AcceptsValidRenamedFile(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_Rejections missing the call to method parallel (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 39 in fbf96f6

func TestSniffPlaylist_Rejections(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestSniffPlaylist_Rejections missing the call to method parallel in test Run (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 51 in fbf96f6

for _, tc := range cases {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_UnverifiedSchemaProceeds missing the call to method parallel (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 69 in fbf96f6

func TestSniffPlaylist_UnverifiedSchemaProceeds(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
integer-format: fmt.Sprintf can be replaced with faster strconv.Itoa (perfsprint)

vbs/cmd/plt.go

Line 211 in fbf96f6

markers = fmt.Sprintf("%d", len(it.Markers))

🚫 [golangci] _{reported by reviewdog 🐶}
parameter name 'p' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt.go

Line 86 in fbf96f6

func resolveInputPath(p string) string {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'it' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt.go

Line 123 in fbf96f6

for _, it := range pl.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'it' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt.go

Line 208 in fbf96f6

for _, it := range view.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'zw' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_fixture_test.go

Line 122 in fbf96f6

zw := zip.NewWriter(&buf)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_fixture_test.go

Line 159 in fbf96f6

db, err := sql.Open("sqlite", dbPath)

🚫 [golangci] _{reported by reviewdog 🐶}
parameter name 'id' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 40 in fbf96f6

func resolveLanguage(id int, override string) (string, error) {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'b' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 57 in fbf96f6

var b strings.Builder

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'b' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 76 in fbf96f6

var b strings.Builder

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'r' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 79 in fbf96f6

for _, r := range s {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'zr' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 146 in fbf96f6

zr, err := zip.OpenReader(path)

🚫 [golangci] _{reported by reviewdog 🐶}
parameter name 'a' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 190 in fbf96f6

func parsePlaylist(a *archive) (*Playlist, error) {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 191 in fbf96f6

db, err := sql.Open("sqlite", a.dbPath)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'it' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 255 in fbf96f6

it Item

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'm' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 359 in fbf96f6

m Marker

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'f' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 374 in fbf96f6

f, err := os.Open(path)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'rc' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 399 in fbf96f6

rc, err := entry.Open()

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'rc' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 434 in fbf96f6

rc, err := entry.Open()

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 457 in fbf96f6

db, err := sql.Open("sqlite", dbPath)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'pl' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse_test.go

Line 37 in fbf96f6

pl := parseFixture(t)

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt.go

Line 58 in fbf96f6

defer func() { _ = arc.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
if statements should only be cuddled with assignments (wsl)

vbs/cmd/plt.go

Line 77 in fbf96f6

if err != nil {

🚫 [golangci] _{reported by reviewdog 🐶}
if statements should only be cuddled with assignments (wsl)

vbs/cmd/plt.go

Line 90 in fbf96f6

if wd := os.Getenv("BUILD_WORKING_DIRECTORY"); wd != "" {

🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

vbs/cmd/plt.go

Line 139 in fbf96f6

view.Items = append(view.Items, pi)

🚫 [golangci] _{reported by reviewdog 🐶}
declarations should never be cuddled (wsl)

vbs/cmd/plt_fixture_test.go

Line 147 in fbf96f6

var buf bytes.Buffer

🚫 [golangci] _{reported by reviewdog 🐶}
ranges should only be cuddled with assignments used in the iteration (wsl)

vbs/cmd/plt_fixture_test.go

Line 217 in fbf96f6

for _, name := range order {

🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

vbs/cmd/plt_fixture_test.go

Line 221 in fbf96f6

stmts = append(stmts, tables[name])

🚫 [golangci] _{reported by reviewdog 🐶}
anonymous switch statements should never be cuddled (wsl)

vbs/cmd/plt_helpers.go

Line 61 in fbf96f6

switch {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_helpers_test.go

Line 112 in fbf96f6

for i := range want {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 288 in fbf96f6

defer func() { _ = rows.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 378 in fbf96f6

defer func() { _ = f.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
ranges should only be cuddled with assignments used in the iteration (wsl)

vbs/cmd/plt_parse.go

Line 482 in fbf96f6

for _, table := range requiredTables {

🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

vbs/cmd/plt_parse_test.go

Line 27 in fbf96f6

t.Cleanup(func() { _ = arc.Close() })

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_parse_test.go

Line 45 in fbf96f6

for i, it := range pl.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_parse_test.go

Line 82 in fbf96f6

for i, want := range wantLabels {

🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

vbs/cmd/plt_sniff_test.go

Line 76 in fbf96f6

t.Cleanup(func() { _ = arc.Close() })

🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (too many statements above range) (wsl_v5)

vbs/cmd/plt_helpers_test.go

Line 111 in fbf96f6

want := []string{"talk", "talk-2", "talk-3", "other"}

🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (invalid statement above assign) (wsl_v5)

vbs/cmd/plt_parse.go

Line 227 in fbf96f6

err := db.QueryRow(`SELECT Name FROM Tag WHERE Type = 2`).Scan(&name)

statik · 2026-06-11T19:05:23Z

+		"TagMap":                  `CREATE TABLE TagMap (TagMapId INTEGER PRIMARY KEY, TagId INTEGER, PlaylistItemId INTEGER, Position INTEGER)`,
+		"PlaylistItem":            `CREATE TABLE PlaylistItem (PlaylistItemId INTEGER PRIMARY KEY, Label TEXT, StartTrimOffsetTicks INTEGER, EndTrimOffsetTicks INTEGER, EndAction INTEGER, ThumbnailFilePath TEXT)`,
+		"PlaylistItemLocationMap": `CREATE TABLE PlaylistItemLocationMap (PlaylistItemId INTEGER, LocationId INTEGER, MajorMultimediaType INTEGER, BaseDurationTicks INTEGER)`,
+		"Location":                `CREATE TABLE Location (LocationId INTEGER PRIMARY KEY, BookNumber INTEGER, ChapterNumber INTEGER, DocumentId INTEGER, Track INTEGER, KeySymbol TEXT, MepsLanguage INTEGER, Type INTEGER)`,


🚫 [golangci] _{reported by reviewdog 🐶}
string Location has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T19:05:23Z

+		"PlaylistItem":            `CREATE TABLE PlaylistItem (PlaylistItemId INTEGER PRIMARY KEY, Label TEXT, StartTrimOffsetTicks INTEGER, EndTrimOffsetTicks INTEGER, EndAction INTEGER, ThumbnailFilePath TEXT)`,
+		"PlaylistItemLocationMap": `CREATE TABLE PlaylistItemLocationMap (PlaylistItemId INTEGER, LocationId INTEGER, MajorMultimediaType INTEGER, BaseDurationTicks INTEGER)`,
+		"Location":                `CREATE TABLE Location (LocationId INTEGER PRIMARY KEY, BookNumber INTEGER, ChapterNumber INTEGER, DocumentId INTEGER, Track INTEGER, KeySymbol TEXT, MepsLanguage INTEGER, Type INTEGER)`,
+		"IndependentMedia":        `CREATE TABLE IndependentMedia (IndependentMediaId INTEGER PRIMARY KEY, OriginalFilename TEXT, FilePath TEXT, MimeType TEXT, Hash TEXT)`,


🚫 [golangci] _{reported by reviewdog 🐶}
string IndependentMedia has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T19:05:23Z

+		"IndependentMedia":        `CREATE TABLE IndependentMedia (IndependentMediaId INTEGER PRIMARY KEY, OriginalFilename TEXT, FilePath TEXT, MimeType TEXT, Hash TEXT)`,
+		"PlaylistItemIndependentMediaMap": `CREATE TABLE PlaylistItemIndependentMediaMap ` +
+			`(PlaylistItemId INTEGER, IndependentMediaId INTEGER, DurationTicks INTEGER)`,
+		"PlaylistItemMarker": `CREATE TABLE PlaylistItemMarker (PlaylistItemMarkerId INTEGER PRIMARY KEY, ` +


🚫 [golangci] _{reported by reviewdog 🐶}
string PlaylistItemMarker has 5 occurrences, make it a constant (goconst)

statik · 2026-06-11T19:05:23Z

+// requiredTables are the database tables the parser depends on. Their presence
+// is the contract that lets us read a playlist regardless of schema version.
+var requiredTables = []string{
+	"Tag",


🚫 [golangci] _{reported by reviewdog 🐶}
string Tag has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T19:05:23Z

+// is the contract that lets us read a playlist regardless of schema version.
+var requiredTables = []string{
+	"Tag",
+	"TagMap",


🚫 [golangci] _{reported by reviewdog 🐶}
string TagMap has 3 occurrences, make it a constant (goconst)

statik · 2026-06-11T19:05:23Z

+	}
+
+	dbPath := filepath.Join(tmpDir, "userData.db")
+	if err := os.WriteFile(dbPath, data, 0o600); err != nil {


🚫 [golangci] _{reported by reviewdog 🐶}
Magic number: 0o600, in detected (mnd)

statik · 2026-06-11T19:05:23Z

+				b.WriteRune(r)
+			}
+			prevHyphen = true
+			continue


🚫 [golangci] _{reported by reviewdog 🐶}
continue with no blank line before (nlreturn)

statik · 2026-06-11T19:05:23Z

+	"testing"
+)
+
+func TestTicksToSeconds(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestTicksToSeconds missing the call to method parallel (paralleltest)

statik · 2026-06-11T19:05:23Z

+		{"image cue 4s", 40000000, 4.0},
+	}
+
+	for _, tc := range cases {


🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestTicksToSeconds missing the call to method parallel in test Run (paralleltest)

statik · 2026-06-11T19:05:23Z

+	}
+}
+
+func TestResolveLanguage(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestResolveLanguage missing the call to method parallel (paralleltest)

statik

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_AcceptsValidRenamedFile missing the call to method parallel (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 22 in 1111b7d

func TestSniffPlaylist_AcceptsValidRenamedFile(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_Rejections missing the call to method parallel (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 39 in 1111b7d

func TestSniffPlaylist_Rejections(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestSniffPlaylist_Rejections missing the call to method parallel in test Run (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 51 in 1111b7d

for _, tc := range cases {

🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_UnverifiedSchemaProceeds missing the call to method parallel (paralleltest)

vbs/cmd/plt_sniff_test.go

Line 69 in 1111b7d

func TestSniffPlaylist_UnverifiedSchemaProceeds(t *testing.T) {

🚫 [golangci] _{reported by reviewdog 🐶}
integer-format: fmt.Sprintf can be replaced with faster strconv.Itoa (perfsprint)

vbs/cmd/plt.go

Line 236 in 1111b7d

markers = fmt.Sprintf("%d", len(it.Markers))

🚫 [golangci] _{reported by reviewdog 🐶}
parameter name 'p' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt.go

Line 82 in 1111b7d

func resolveInputPath(p string) string {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'it' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt.go

Line 148 in 1111b7d

for _, it := range pl.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'it' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt.go

Line 233 in 1111b7d

for _, it := range view.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'zw' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_fixture_test.go

Line 122 in 1111b7d

zw := zip.NewWriter(&buf)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_fixture_test.go

Line 159 in 1111b7d

db, err := sql.Open("sqlite", dbPath)

🚫 [golangci] _{reported by reviewdog 🐶}
parameter name 'id' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 40 in 1111b7d

func resolveLanguage(id int, override string) (string, error) {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'b' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 57 in 1111b7d

var b strings.Builder

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'b' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 76 in 1111b7d

var b strings.Builder

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'r' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_helpers.go

Line 79 in 1111b7d

for _, r := range s {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'zr' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 146 in 1111b7d

zr, err := zip.OpenReader(path)

🚫 [golangci] _{reported by reviewdog 🐶}
parameter name 'a' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 190 in 1111b7d

func parsePlaylist(a *archive) (*Playlist, error) {

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 191 in 1111b7d

db, err := sql.Open("sqlite", a.dbPath)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'it' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 255 in 1111b7d

it Item

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'm' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 359 in 1111b7d

m Marker

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'f' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 374 in 1111b7d

f, err := os.Open(path)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'rc' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 399 in 1111b7d

rc, err := entry.Open()

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'rc' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 434 in 1111b7d

rc, err := entry.Open()

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse.go

Line 457 in 1111b7d

db, err := sql.Open("sqlite", dbPath)

🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'pl' is too short for the scope of its usage (varnamelen)

vbs/cmd/plt_parse_test.go

Line 37 in 1111b7d

pl := parseFixture(t)

🚫 [golangci] _{reported by reviewdog 🐶}
if statements should only be cuddled with assignments (wsl)

vbs/cmd/plt.go

Line 73 in 1111b7d

if err != nil {

🚫 [golangci] _{reported by reviewdog 🐶}
if statements should only be cuddled with assignments (wsl)

vbs/cmd/plt.go

Line 86 in 1111b7d

if wd := os.Getenv("BUILD_WORKING_DIRECTORY"); wd != "" {

🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

vbs/cmd/plt.go

Line 164 in 1111b7d

view.Items = append(view.Items, pi)

🚫 [golangci] _{reported by reviewdog 🐶}
declarations should never be cuddled (wsl)

vbs/cmd/plt_fixture_test.go

Line 147 in 1111b7d

var buf bytes.Buffer

🚫 [golangci] _{reported by reviewdog 🐶}
ranges should only be cuddled with assignments used in the iteration (wsl)

vbs/cmd/plt_fixture_test.go

Line 217 in 1111b7d

for _, name := range order {

🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

vbs/cmd/plt_fixture_test.go

Line 221 in 1111b7d

stmts = append(stmts, tables[name])

🚫 [golangci] _{reported by reviewdog 🐶}
anonymous switch statements should never be cuddled (wsl)

vbs/cmd/plt_helpers.go

Line 61 in 1111b7d

switch {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_helpers_test.go

Line 112 in 1111b7d

for i := range want {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 354 in 1111b7d

defer func() { _ = rows.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 403 in 1111b7d

defer func() { _ = rc.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 438 in 1111b7d

defer func() { _ = rc.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
ranges should only be cuddled with assignments used in the iteration (wsl)

vbs/cmd/plt_parse.go

Line 482 in 1111b7d

for _, table := range requiredTables {

🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

vbs/cmd/plt_parse_test.go

Line 27 in 1111b7d

t.Cleanup(func() { _ = arc.Close() })

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_parse_test.go

Line 45 in 1111b7d

for i, it := range pl.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_parse_test.go

Line 82 in 1111b7d

for i, want := range wantLabels {

🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

vbs/cmd/plt_sniff_test.go

Line 76 in 1111b7d

t.Cleanup(func() { _ = arc.Close() })

🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (too many statements above range) (wsl_v5)

vbs/cmd/plt_helpers_test.go

Line 111 in 1111b7d

want := []string{"talk", "talk-2", "talk-3", "other"}

🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (invalid statement above assign) (wsl_v5)

vbs/cmd/plt_parse.go

Line 227 in 1111b7d

err := db.QueryRow(`SELECT Name FROM Tag WHERE Type = 2`).Scan(&name)

statik · 2026-06-11T20:24:03Z

+func checkPlaylistFile(path string) error {
+	if _, err := os.Stat(path); err != nil {
+		if errors.Is(err, os.ErrNotExist) {
+			return fmt.Errorf("no such file: %s", path)


🚫 [golangci] _{reported by reviewdog 🐶}
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf("no such file: %s", path)" (err113)

statik · 2026-06-11T20:24:03Z

+
+var pltPrintJSON bool
+
+var pltCmd = &coral.Command{


🚫 [golangci] _{reported by reviewdog 🐶}
coral.Command is missing fields Aliases, SuggestFor, ValidArgs, ValidArgsFunction, Args, ArgAliases, BashCompletionFunction, Deprecated, Annotations, Version, PersistentPreRun, PersistentPreRunE, PreRun, PreRunE, Run, RunE, PostRun, PostRunE, PersistentPostRun, PersistentPostRunE, FParseErrWhitelist, CompletionOptions, TraverseChildren, Hidden, SilenceErrors, SilenceUsage, DisableFlagParsing, DisableAutoGenTag, DisableFlagsInUseLine, DisableSuggestions, SuggestionsMinimumDistance (exhaustruct)

statik · 2026-06-11T20:24:04Z

+  vbs plt build meeting.playlist`,
+}
+
+var pltPrintCmd = &coral.Command{


🚫 [golangci] _{reported by reviewdog 🐶}
coral.Command is missing fields Aliases, SuggestFor, ValidArgs, ValidArgsFunction, ArgAliases, BashCompletionFunction, Deprecated, Annotations, Version, PersistentPreRun, PersistentPreRunE, PreRun, PreRunE, RunE, PostRun, PostRunE, PersistentPostRun, PersistentPostRunE, FParseErrWhitelist, CompletionOptions, TraverseChildren, Hidden, SilenceErrors, SilenceUsage, DisableFlagParsing, DisableAutoGenTag, DisableFlagsInUseLine, DisableSuggestions, SuggestionsMinimumDistance (exhaustruct)

statik · 2026-06-11T20:24:04Z

+	view := printView{Name: pl.Name, Items: make([]printItem, 0, len(pl.Items))}
+
+	for _, it := range pl.Items {
+		pi := printItem{


🚫 [golangci] _{reported by reviewdog 🐶}
cmd.printItem is missing field Markers (exhaustruct)

statik · 2026-06-11T20:24:04Z

+	}{
+		{
+			"pub track",
+			Item{Location: &Location{KeySymbol: "sjj", Track: 135, Type: 0}},


🚫 [golangci] _{reported by reviewdog 🐶}
cmd.Item is missing fields Position, PlaylistItemID, Label, StartTrimTicks, EndTrimTicks, EndAction, ThumbnailPath, Image, Markers (exhaustruct)

statik · 2026-06-11T20:24:04Z

+	}
+}
+
+func TestDescribeSource(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestDescribeSource missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:24:04Z

+		},
+	}
+
+	for _, tc := range cases {


🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestDescribeSource missing the call to method parallel in test Run (paralleltest)

statik · 2026-06-11T20:24:04Z

+	}
+}
+
+func TestBuildPrintView(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestBuildPrintView missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:24:04Z

+	}
+}
+
+func TestRenderJSON_RoundTrips(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestRenderJSON_RoundTrips missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:24:05Z

+	}
+}
+
+func TestRenderText_ContainsKeyData(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestRenderText_ContainsKeyData missing the call to method parallel (paralleltest)

statik

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci

🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

vbs/cmd/plt_fixture_test.go

Line 221 in b48cb2f

stmts = append(stmts, tables[name])

🚫 [golangci] _{reported by reviewdog 🐶}
anonymous switch statements should never be cuddled (wsl)

vbs/cmd/plt_helpers.go

Line 61 in b48cb2f

switch {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_helpers_test.go

Line 112 in b48cb2f

for i := range want {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 150 in b48cb2f

defer func() { _ = zr.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 248 in b48cb2f

defer func() { _ = rows.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

vbs/cmd/plt_parse.go

Line 438 in b48cb2f

defer func() { _ = rc.Close() }()

🚫 [golangci] _{reported by reviewdog 🐶}
ranges should only be cuddled with assignments used in the iteration (wsl)

vbs/cmd/plt_parse.go

Line 482 in b48cb2f

for _, table := range requiredTables {

🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

vbs/cmd/plt_parse_test.go

Line 27 in b48cb2f

t.Cleanup(func() { _ = arc.Close() })

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_parse_test.go

Line 45 in b48cb2f

for i, it := range pl.Items {

🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

vbs/cmd/plt_parse_test.go

Line 82 in b48cb2f

for i, want := range wantLabels {

🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

vbs/cmd/plt_sniff_test.go

Line 29 in b48cb2f

t.Cleanup(func() { _ = arc.Close() })

🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (too many statements above range) (wsl_v5)

vbs/cmd/plt_helpers_test.go

Line 111 in b48cb2f

want := []string{"talk", "talk-2", "talk-3", "other"}

🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (invalid statement above assign) (wsl_v5)

vbs/cmd/plt_parse.go

Line 227 in b48cb2f

err := db.QueryRow(`SELECT Name FROM Tag WHERE Type = 2`).Scan(&name)

statik · 2026-06-11T20:45:12Z

+	}
+}
+
+func TestParsePlaylist_BookChapterMarkers(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestParsePlaylist_BookChapterMarkers missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:45:12Z

+	"testing"
+)
+
+func TestSniffPlaylist_AcceptsValidRenamedFile(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_AcceptsValidRenamedFile missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:45:12Z

+	}
+}
+
+func TestSniffPlaylist_Rejections(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_Rejections missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:45:12Z

+		{"missing required table", fixtureOptions{omitTables: []string{"PlaylistItemMarker"}}, "PlaylistItemMarker"},
+	}
+
+	for _, tc := range cases {


🚫 [golangci] _{reported by reviewdog 🐶}
Range statement for test TestSniffPlaylist_Rejections missing the call to method parallel in test Run (paralleltest)

statik · 2026-06-11T20:45:12Z

+	}
+}
+
+func TestSniffPlaylist_UnverifiedSchemaProceeds(t *testing.T) {


🚫 [golangci] _{reported by reviewdog 🐶}
Function TestSniffPlaylist_UnverifiedSchemaProceeds missing the call to method parallel (paralleltest)

statik · 2026-06-11T20:45:13Z

+
+// verifyTables confirms every required table is present in the database.
+func verifyTables(dbPath string) error {
+	db, err := sql.Open("sqlite", dbPath)


🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'db' is too short for the scope of its usage (varnamelen)

statik · 2026-06-11T20:45:13Z

+}
+
+func TestParsePlaylist_NameAndOrder(t *testing.T) {
+	pl := parseFixture(t)


🚫 [golangci] _{reported by reviewdog 🐶}
variable name 'pl' is too short for the scope of its usage (varnamelen)

statik · 2026-06-11T20:45:13Z

+				DurationSec: ticksToSeconds(m.DurationTicks),
+			})
+		}
+		view.Items = append(view.Items, pi)


🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

statik · 2026-06-11T20:45:13Z

+	t.Helper()
+
+	img := image.NewRGBA(image.Rect(0, 0, 1, 1))
+	var buf bytes.Buffer


🚫 [golangci] _{reported by reviewdog 🐶}
declarations should never be cuddled (wsl)

statik · 2026-06-11T20:45:13Z

+	}
+
+	var stmts []string
+	for _, name := range order {


🚫 [golangci] _{reported by reviewdog 🐶}
ranges should only be cuddled with assignments used in the iteration (wsl)

statik · 2026-06-12T16:01:26Z

+	if it.IsImage() {
+		return ticksToSeconds(it.Image.DurationTicks)
+	}
+	if it.Location != nil {


🚫 [golangci] _{reported by reviewdog 🐶}
if statements should only be cuddled with assignments (wsl)

statik · 2026-06-12T16:01:26Z

+		if omit[name] {
+			continue
+		}
+		stmts = append(stmts, tables[name])


🚫 [golangci] _{reported by reviewdog 🐶}
append only allowed to cuddle with appended value (wsl)

statik · 2026-06-12T16:01:26Z

+
+	for _, r := range s {
+		r = unicode.ToLower(r)
+		switch {


🚫 [golangci] _{reported by reviewdog 🐶}
anonymous switch statements should never be cuddled (wsl)

statik · 2026-06-12T16:01:26Z

+		uniqueSlug("other", seen),
+	}
+	want := []string{"talk", "talk-2", "talk-3", "other"}
+	for i := range want {


🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

statik · 2026-06-12T16:01:26Z

+	if err != nil {
+		return fmt.Errorf("could not open database: %w", err)
+	}
+	defer func() { _ = db.Close() }()


🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

statik · 2026-06-12T16:01:26Z

+	if len(pl.Items) != 4 {
+		t.Fatalf("len(Items) = %d, want 4", len(pl.Items))
+	}
+	for i, it := range pl.Items {


🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

statik · 2026-06-12T16:01:26Z

+		t.Fatalf("chapter markers = %d, want 3", len(chapter.Markers))
+	}
+	wantLabels := []string{"Marker one", "Marker two", "Marker three"}
+	for i, want := range wantLabels {


🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before range statement (wsl)

statik · 2026-06-12T16:01:26Z

+	if err != nil {
+		t.Fatalf("unverified schema version should still parse, got: %v", err)
+	}
+	t.Cleanup(func() { _ = arc.Close() })


🚫 [golangci] _{reported by reviewdog 🐶}
expressions should not be cuddled with blocks (wsl)

statik · 2026-06-12T16:01:27Z

+		uniqueSlug("talk", seen),
+		uniqueSlug("other", seen),
+	}
+	want := []string{"talk", "talk-2", "talk-3", "other"}


🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (too many statements above range) (wsl_v5)

statik · 2026-06-12T16:01:27Z

+// queryPlaylistName returns the name of the playlist tag (Type 2).
+func queryPlaylistName(db *sql.DB) (string, error) {
+	var name string
+	err := db.QueryRow(`SELECT Name FROM Tag WHERE Type = 2`).Scan(&name)


🚫 [golangci] _{reported by reviewdog 🐶}
missing whitespace above this line (invalid statement above assign) (wsl_v5)

Parse purple playlist exports (ZIP + SQLite) and print their cues as a table or JSON, fully offline. Includes a content-based format sniffer that rejects invalid files with named reasons, a SQLite-to-domain parser for the verified table contract, and a synthetic fixture generator so tests never depend on real exports. Promotes modernc.org/sqlite to a direct dependency (pure-Go driver, no CGO) and adds a vendor-neutrality CI guard that rejects the publisher's identifiers in authored files.

… run Add an Examples section and a descriptive Use string so 'vbs plt --help' makes clear each subcommand takes a playlist file. Resolve relative input paths against BUILD_WORKING_DIRECTORY so 'bazel run' works from the directory the user invoked it in, not the runfiles tree.

A missing or unreadable path now fails with "Could not open playlist file: no such file" instead of the format-validation message, which was misleading for a mistyped path. Extracted openPlaylist as the shared resolve-check-sniff entry point for the print and build commands.

statik · 2026-06-12T20:06:07Z

+	files := map[string][]byte{
+		dbName:            dbBytes,
+		fixtureThumbA:     onePixelJPEG(t),
+		fixtureThumbBook: onePixelJPEG(t),


🚫 [golangci] _{reported by reviewdog 🐶}
File is not properly formatted (gci)

statik · 2026-06-12T20:06:07Z

+	} else {
+		err = renderText(os.Stdout, view)
+	}
+	if err != nil {


🚫 [golangci] _{reported by reviewdog 🐶}
if statements should only be cuddled with assignments (wsl)

statik · 2026-06-12T20:06:07Z

+	if err != nil {
+		return fmt.Errorf("could not read embedded media: %w", err)
+	}
+	defer func() { _ = rows.Close() }()


🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

statik · 2026-06-12T20:06:07Z

+	if err != nil {
+		return fmt.Errorf("could not list database tables: %w", err)
+	}
+	defer func() { _ = rows.Close() }()


🚫 [golangci] _{reported by reviewdog 🐶}
only one cuddle assignment allowed before defer statement (wsl)

statik added the feature label Jun 11, 2026

statik commented Jun 11, 2026

View reviewed changes

statik mentioned this pull request Jun 11, 2026

Add plt build command to download and cut playlist media #887

Merged

6 tasks

statik commented Jun 11, 2026

View reviewed changes

statik force-pushed the plt-phase-1 branch from fbf96f6 to 1111b7d Compare June 11, 2026 20:23

statik commented Jun 11, 2026

View reviewed changes

statik force-pushed the plt-phase-1 branch from b48cb2f to 01aa59b Compare June 12, 2026 16:00

statik commented Jun 12, 2026

View reviewed changes

statik added 3 commits June 12, 2026 16:02

statik force-pushed the plt-phase-1 branch from 01aa59b to eb704f8 Compare June 12, 2026 20:05

statik commented Jun 12, 2026

View reviewed changes

statik merged commit 1441cc8 into main Jun 12, 2026
13 checks passed

statik deleted the plt-phase-1 branch June 12, 2026 20:12

Uh oh!

Conversation

statik commented Jun 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test Plan

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

statik left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

statik left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

statik left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

statik commented Jun 11, 2026 •

edited

Loading