audio streaming app
plyr.fm
fix: tie costs export to production release tag (#1166)
the hourly costs export was running from main, so schema changes
(like audd → copyright_scanning in #1163) broke the prod /costs
page before the frontend was released.
- checkout latest release tag in export-costs.yml
- add CI-only guard to export_costs.py (refuse non-dry-run locally)
- add pre-commit hook to catch stale privacy policy dates
- filed #1165 for moderation staging environment
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
authored by
nate nowack
Claude Opus 4.6
and committed by
GitHub
f8fa8ff0
0e2c493d
+107
+8
.github/workflows/export-costs.yml
+8
.github/workflows/export-costs.yml
···
31
31
runs-on: ubuntu-latest
32
32
33
33
steps:
34
+
- name: get latest release tag
35
+
id: release
36
+
run: echo "tag=$(gh release view --json tagName -q .tagName)" >> "$GITHUB_OUTPUT"
37
+
env:
38
+
GH_TOKEN: ${{ github.token }}
39
+
34
40
- uses: actions/checkout@v4
41
+
with:
42
+
ref: ${{ steps.release.outputs.tag }}
35
43
36
44
- uses: astral-sh/setup-uv@v4
37
45
+7
.pre-commit-config.yaml
+7
.pre-commit-config.yaml
···
48
48
files: ^frontend/.*\.(ts|svelte)$
49
49
pass_filenames: false
50
50
51
+
- id: check-legal-dates
52
+
name: check legal dates
53
+
entry: python scripts/check_legal_dates.py
54
+
language: system
55
+
files: ^frontend/src/routes/privacy/
56
+
pass_filenames: true
57
+
51
58
- id: cargo-check-moderation
52
59
name: cargo check (moderation)
53
60
entry: bash -c 'cd services/moderation && cargo check --quiet'
+87
scripts/check_legal_dates.py
+87
scripts/check_legal_dates.py
···
1
+
#!/usr/bin/env python3
2
+
"""pre-commit hook: ensure privacy policy date is updated when content changes."""
3
+
4
+
import re
5
+
import subprocess
6
+
import sys
7
+
from datetime import datetime
8
+
9
+
DATE_PATTERN = re.compile(r"Last updated:\s*(\w+ \d{1,2},\s*\d{4})")
10
+
11
+
12
+
def get_staged_content(path: str) -> str:
13
+
result = subprocess.run(
14
+
["git", "show", f":{path}"],
15
+
capture_output=True,
16
+
text=True,
17
+
)
18
+
return result.stdout
19
+
20
+
21
+
def get_staged_diff(path: str) -> str:
22
+
result = subprocess.run(
23
+
["git", "diff", "--cached", "--", path],
24
+
capture_output=True,
25
+
text=True,
26
+
)
27
+
return result.stdout
28
+
29
+
30
+
def check_file(path: str) -> bool:
31
+
content = get_staged_content(path)
32
+
if not content:
33
+
return True
34
+
35
+
match = DATE_PATTERN.search(content)
36
+
if not match:
37
+
print(f" {path}: no 'Last updated' date found")
38
+
return False
39
+
40
+
date_str = match.group(1)
41
+
try:
42
+
doc_date = datetime.strptime(date_str, "%B %d, %Y")
43
+
except ValueError:
44
+
print(f" {path}: could not parse date '{date_str}'")
45
+
return False
46
+
47
+
diff = get_staged_diff(path)
48
+
diff_lines = diff.splitlines()
49
+
50
+
# check if there are content changes beyond just the date line
51
+
content_changed = False
52
+
for line in diff_lines:
53
+
if not line.startswith(("+", "-")):
54
+
continue
55
+
if line.startswith(("+++", "---")):
56
+
continue
57
+
if "Last updated:" in line:
58
+
continue
59
+
# ignore pure whitespace changes
60
+
stripped = line[1:].strip()
61
+
if stripped:
62
+
content_changed = True
63
+
break
64
+
65
+
if not content_changed:
66
+
return True
67
+
68
+
now = datetime.now()
69
+
if doc_date.year == now.year and doc_date.month == now.month:
70
+
return True
71
+
72
+
print(f" {path}: privacy policy content changed — update the 'Last updated' date")
73
+
print(f" current: {date_str}, expected: current month ({now.strftime('%B %Y')})")
74
+
return False
75
+
76
+
77
+
def main() -> int:
78
+
files = sys.argv[1:]
79
+
if not files:
80
+
return 0
81
+
82
+
ok = all(check_file(f) for f in files)
83
+
return 0 if ok else 1
84
+
85
+
86
+
if __name__ == "__main__":
87
+
sys.exit(main())
+5
scripts/costs/export_costs.py
+5
scripts/costs/export_costs.py
···
13
13
14
14
import asyncio
15
15
import json
16
+
import os
16
17
import re
17
18
from datetime import UTC, datetime, timedelta
18
19
from typing import Any
···
213
214
env: str = typer.Option("prd", "--env", "-e", help="environment: prd, stg, dev"),
214
215
) -> None:
215
216
"""export platform costs to R2 for public dashboard"""
217
+
if not os.environ.get("CI") and not dry_run:
218
+
typer.echo("costs export should only run in CI (GitHub Actions)")
219
+
typer.echo(" use --dry-run for local testing")
220
+
raise typer.Exit(1)
216
221
217
222
async def run():
218
223
db_url = settings.get_db_url(env)