Ticket graphs

Every ticket as the root of its provenance graph — goal, patch, approval, runs and (as the ticket-class build lands) mandate, manifestations, guarantees, journey, artifacts and children. Rendered through the <<<TicketGraphList>>> tag.

← all ticket-graphs

DONE#347 forge-refactor

Rewrite audio_tagger:audio_tagger/tagger.py as behavior-preserving, installable, control-config'd Python [forge-rw:12747b17a9a8e06d]

goal

THIS UNIT SUPERSEDES #260 — it is the SAME module and the same work, re-filed to be SELF-CONTAINED. The superseded unit named its source only as a `git cat-file` command against the forge's bare repos, which the sandboxed pair_loop worker that executes these tickets cannot run, so it could not be done as filed (#346; #259 is the proof). A ticket body is permanent in SQL (tg_tickets_immutable_identity, migrations/0029_ticket_authorization .sql:111-117), so that body could not be repaired — this ticket carries the fix instead, and the original is halted by a stop ticket rather than erased. Do the work HERE. Rewrite ONE unique forge module as entirely new, behavior-preserving, installable, control-config'd Python, per the FORGE-wide refactor standard #200 (#std). This ticket covers ONE module — the one identified below — and nothing else. It was filed against the module's DEDUPLICATED identity, so if that module was copy-pasted across the forge, every copy is covered by this single ticket. MODULE KEY : forge-rw:12747b17a9a8e06d (the ticketer's idempotency marker — it lives in this ticket's TITLE. Do not edit the title, or the module will be re-filed as a duplicate.) IDENTITY : AST-NORMALIZED — codebean's `_ast_norm` (#183), i.e. ast.dump(ast.parse(ast.unparse(ast.parse(src)))). Copies of this module that differ only in comments, blank lines, quoting or spacing carry the SAME module-sha and are covered by THIS ticket. A differing docstring is real content, so a module whose docstring differs is a DIFFERENT module with its own ticket. SIZE : 3063 bytes EXTENSION : .py CANONICAL SOURCE (the location this ticket is named for): repo : audio_tagger path : audio_tagger/tagger.py ref : HEAD (as recorded when this module was first filed) blob : 1218ce1b5c59d876d8b27aa83b2cc99fdba4f5db (git blob sha1) REFERENCE ONLY — you do NOT need this command. This module's bytes are embedded COMPLETE in THE ORIGINAL SOURCE block below, and that is what the rewrite is based on. For a reader who does have forge access: git --git-dir=/Users/stevenpeterson/code/installed/jazz-project/config/forge/remotes/audio_tagger.git cat-file blob 1218ce1b5c59d876d8b27aa83b2cc99fdba4f5db THE SAME MODULE ALSO LIVES AT THESE FORGE TIPS — 4 copy/copies. They SHARE THIS ticket: this module is rewritten ONCE, and every location below is satisfied by that one rewrite. Do not file a ticket per copy. code__code__jazz__kokoro__audio_tagger:audio_tagger/tagger.py gitlab__testmonkeyalpha-group__jazz-forge__audio_tagger:audio_tagger/tagger.py gitlab__testmonkeyalpha__audio_tagger:audio_tagger/tagger.py testmonkeyalpha-group__jazz-forge__audio_tagger:audio_tagger/tagger.py HISTORICAL OCCURRENCES of this exact blob — every OTHER DISTINCT {repo, path} the omni-git index (`occ`, joined on the git blob sha1) records this byte-identical file at, with the index's own `ts` for the first row of each. Any place this run enumerated LIVE is listed under the forge tips above, not here. Context for the rewrite, not extra work: testmonkeyalpha/audio_tagger:audio_tagger/tagger.py [occ.ts gitlab] THE ORIGINAL SOURCE, EMBEDDED — YOU NEED NO FORGE ACCESS TO DO THIS TICKET. The module's own bytes are reproduced below, inside this ticket. THEY are the authoritative copy and the behavior-preserving basis: rewrite from them. The `git --git-dir=... cat-file blob` command under CANONICAL SOURCE above is kept only as a REFERENCE for a reader who happens to have forge access — running it is NOT part of this ticket. (Why the bytes are here at all: the pair_loop worker that executes this ticket is sandboxed to the ticket's own congruency worktree and cannot read the forge's bare repos. Ticket #346 is the fix that put them on the ticket.) The block BEGINS at the line ===ORIGINAL SOURCE (verbatim, the behavior-preserving basis)=== and ENDS at the first line ===END SOURCE=== Everything strictly between those two lines is the module — 3063 byte(s), byte for byte, with no re-indentation, no re-wrapping and nothing elided. ===ORIGINAL SOURCE (verbatim, the behavior-preserving basis)=== # SPDX-License-Identifier: Apache-2.0 # Copyright 2026 testmonkeyalpha — Claude & Co. import os import json from mutagen.mp3 import MP3 from mutagen.id3 import ID3, TIT2, COMM, USLT, error class AudioTagger: def __init__(self, notes_file: str = "notes.json", mp3_dir: str = None): self.notes_file = notes_file self.mp3_dir = mp3_dir or os.getcwd() def run(self): if not os.path.exists(self.notes_file): print(f"Error: '{self.notes_file}' not found.") return with open(self.notes_file, "r", encoding="utf-8") as f: module_data = json.load(f) module_name = module_data.get("name", "Unknown Module") print(f"Tagging MP3s for module: {module_name}") for obj_node in module_data.get("children", []): if obj_node.get("type") == "objective": obj_name = obj_node.get("name", "Unknown Objective") for note in obj_node.get("children", []): if note.get("type") == "note": note_id = note.get("id") note_name = note.get("name") text_content = note.get("metadata", {}).get("text_content", note_name) file_path = os.path.join(self.mp3_dir, f"{note_id}.mp3") self._tag(file_path, note_name, text_content, module_name, obj_name) print("Done.") def tag_note(self, node_id: str): if not os.path.exists(self.notes_file): return with open(self.notes_file, "r", encoding="utf-8") as f: module_data = json.load(f) module_name = module_data.get("name", "Unknown Module") for obj_node in module_data.get("children", []): if obj_node.get("type") == "objective": obj_name = obj_node.get("name", "Unknown Objective") for note in obj_node.get("children", []): if note.get("type") == "note" and note.get("id") == node_id: file_path = os.path.join(self.mp3_dir, f"{node_id}.mp3") self._tag(file_path, note.get("name"), note.get("metadata", {}).get("text_content", ""), module_name, obj_name) return def _tag(self, file_path: str, title: str, text: str, module_name: str, obj_name: str): if not os.path.exists(file_path): return try: audio = MP3(file_path, ID3=ID3) try: audio.add_tags() except error: pass audio.tags.add(TIT2(encoding=3, text=title)) audio.tags.add(COMM(encoding=3, lang='eng', desc='Lineage', text=f"{module_name} -> {obj_name}")) audio.tags.add(USLT(encoding=3, lang='eng', desc='', text=text)) audio.save() print(f" tagged: {os.path.basename(file_path)}") except Exception as e: print(f" error tagging {os.path.basename(file_path)}: {e}") if __name__ == "__main__": AudioTagger().run() ===END SOURCE=== THE CALLABLE SURFACE — every callable in the original, lowered with codebean's `lower_node` (#183). The rewrite must present THIS surface (same names, same parameters, same defaults) so callers of the original keep working: AudioTagger.__init__(self, notes_file: str='notes.json', mp3_dir: str=None) AudioTagger.run(self) AudioTagger.tag_note(self, node_id: str) AudioTagger._tag(self, file_path: str, title: str, text: str, module_name: str, obj_name: str) CAPTURED BEHAVIORAL EXEMPLARS: none. no function in this module is minable — criteria_miner mines only self-contained pure functions (it refuses anything that mentions a world-touching or non-deterministic name, plus methods and no-arg functions), and nothing here qualified This does NOT weaken the ticket's bar — it MOVES the work: you must write the equivalence exemplars by hand from the original's behavior before rewriting, and they are what settles this ticket. THE STANDARD — #200's four requirements, which this ticket exists to enforce. All four are the standard's own words; none is optional: 1. BEHAVIOR-PRESERVING, VERIFIED NOT ASSERTED — the deliverable is ENTIRELY NEW source code (rewritten, not copied). It must reproduce the ORIGINAL's behavior, and the proof is the exemplars above (captured via codebean lower_* #183 + criteria_miner IO-exemplars, exactly as #200 specifies). 2. A PYTHON MODULE — regardless of the source language. This module's source is `.py`; the rewrite is Python either way. 3. INSTALLABLE — a proper package: pyproject.toml, `pip install -e .` works, console entry points where applicable. 4. CONTROL-CONFIG'D — NO hardcoded paths or params. A JSON/registry config object drives it (the install.json / registry.json idiom of this repo), and per note-for-claude a tool that cannot see its config THROWS rather than guessing a default. ACCEPTANCE (#200's, per rewrite ticket) — this ticket settles when ALL of: a. the new source reproduces every captured exemplar above, plus the edge cases you add for what sampling cannot reach (paste the runs as evidence); b. it presents the callable surface above, so existing callers keep working; c. `pip install -e .` installs it clean from its own pyproject.toml; d. it runs off a config object with ZERO hardcoded settings, and throws without one. Kind is `build` (a pair: coder + adversarial tester). This ticket hangs off #200 by a `spawned` edge, so a stop ticket aimed at #200 brakes every rewrite in this programme. Filed mechanically by checkouts/current/congruency/tools/forge_refactor_ticketer.py under ticket #201, from the forge bare repos at /Users/stevenpeterson/code/installed/jazz-project/config/forge/remotes. Its original source is embedded above (#346), so this ticket is complete on its own.

patch

none

approval

unapproved

runs

no runs recorded

mandate (clauses)

not yet recorded — lands with the ticket-class build

manifestations

not yet recorded — lands with the ticket-class build

guarantees

not yet recorded — lands with the ticket-class build

journey (blunders & successes)

not yet recorded — lands with the ticket-class build

artifacts (forge)

not yet recorded — lands with the ticket-class build

source (forge tree)

browse congruency source (forge-parked tree)

children

#260 Rewrite audio_tagger:audio_tagger/tagger.py as behavior-preserving, installable, control-config'd Python [forge-rw:12747b17a9a8e06d] (supersedes · REQUESTED)