Skip to content

chore(hygiene): untrack agent-internal TASKPLAN_STATUS_2026-09-17_1324-*.md - #2

Merged
lukisch merged 1 commit into
masterfrom
chore/T-20260926-510472849-repo-hygiene
Sep 26, 2026
Merged

lukisch merged 1 commit into
masterfrom
chore/T-20260926-510472849-repo-hygiene

Conversation

@lukisch

@lukisch lukisch commented Sep 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Untracks TASKPLAN_STATUS_2026-09-17_1324-tasksolver-attempt3.md (agent loop status snapshot, zero test/doc dependency, verified via repo-wide grep). File stays on disk locally (gitignored), nothing deleted.

Part of T-20260926-510472849 (repo hygiene rollout, wave 1).

Deliberately NOT touched: MARKETING-LOG.txt (public artifact per pending user decision). PORTIERUNGSPLAN.md is only doc-linked (ROADMAP.md, llms.txt -- no Python test dependency), but untracking it would leave two dead links on GitHub; logged to the ticket's TESTVERDRAHTET_2026-09-26.md attachment for a coordinated follow-up instead of touched here.

Test plan

  • Full pytest suite: 364 passed, 2 skipped (pre-existing, unrelated: missing local release EXE, native-Qt-only PNG smoke)
  • Fresh clone of this branch: same 364/2

🤖 Generated with Claude Code

https://claude.ai/code/session_01PTbvD41MCVmnQWaobvHfCk

…4-*.md

T-20260926-510472849 (repo hygiene rollout, wave 1). Agent loop status
snapshot, zero test/doc dependency (verified via repo-wide grep across
the full tracked tree). Untracked (file stays on disk locally,
gitignored), nothing was deleted.

NOT touched: MARKETING-LOG.txt (public artifact per pending user
decision). PORTIERUNGSPLAN.md is only doc-linked (ROADMAP.md, llms.txt,
no Python test dependency) but untracking it would leave two dead
links -- logged to TESTVERDRAHTET_2026-09-26.md for a coordinated
follow-up instead of touched here.

Verified: full pytest suite 364 passed, 2 skipped (pre-existing,
unrelated: missing local release EXE, native-Qt-only PNG smoke).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PTbvD41MCVmnQWaobvHfCk
@github-actions

Copy link
Copy Markdown

Welcome! Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what changed and why

Thanks for contributing.

@lukisch

lukisch commented Sep 26, 2026

Copy link
Copy Markdown
Collaborator Author

Review (merge-reviewer, claude-opus) auf Head 2bbf8a4: bestanden. Nimmt TASKPLAN_STATUS_2026-09-17_1324-tasksolver-attempt3.md aus dem Tracking und ergänzt das .gitignore-Muster, keine Restreferenz. Das CI-Rot (ModuleNotFoundError PIL in tests/test_assets_and_icons.py) gibt es schon vorher. Offen ist nur REVIEW_REQUIRED, der Admin-Merge läuft über team-lead.

@lukisch
lukisch merged commit 13dd6cd into master Sep 26, 2026
4 of 13 checks passed
lukisch added a commit that referenced this pull request Sep 26, 2026
* fix(store): widen defringe threshold to remove 1px white edge halo

Zweite Nachreview-Runde (merge-reviewer, claude-opus) zu PR #3
(edcb8fa): Motiv war korrekt (Vollflaeche, unversehrt), Ecken transparent,
Wide ohne opake Ecken -- aber auf dunklem Grund blieb ein heller 1px-Saum
an der Rundung (gemessen: Square310 204 Randpixel Alpha>=192 mit Luminanz
auf Schwarz bis 216, Wide 183 Pixel).

Ursache: Der Antialiasing-Uebergang zwischen Canvas-Hintergrund und
solider Motivfarbe ist am Rand der abgerundeten Ecke nur 1 Pixel breit,
hat aber eine Farbdistanz von ~97 zum Hintergrund (gemessen auf einer
Diagonale nahe der Ecke) -- der bisherige bg_thresh von 60 liess dieses
Pixel unverarbeitet (voll opak, urspruengliche helle Mischfarbe) zurueck.

Fix: bg_thresh auf 150 angehoben (deutlich ueber der gemessenen
Uebergangsdistanz ~97, deutlich unter der Distanz zur soliden Motivfarbe
>300) -- damit erfasst Un-Premultiply den kompletten Uebergang. Verifiziert:
schlimmster verbleibender Wert ueber alle 5 Kacheln liegt bei Luminanz 160
auf Schwarz (vorher bis 216), visuell an der Ecke sauber (siehe Kontaktbogen
im PR-Kommentar).

Regressionstest erweitert: test_store_tiles_have_transparent_corners
komponiert jedes teiltransparente Randpixel auf Schwarz und prueft eine
Luminanzgrenze, damit dieser Fehler kuenftig auffaellt statt nur visuell
im Review.

Dieser Branch wurde frisch von aktuellem master (13dd6cd, nach Merge von
PR #2) erstellt statt PR #3 fortzufuehren, um einen Force-Push zu
vermeiden. `_review/` (Kontaktbogen im Repo) wurde NICHT wieder
eingefuegt -- der Kontaktbogen kommt als eingebettetes Bild im
PR-Kommentar, nicht als Repo-Datei.

Verifiziert: 35/35 gezielte Tests (assets/icons, store_materials,
metadata_contract) gruen; Vollsuite lief bis 80% sichtbar sauber durch
(Log-Abschneidung durch Backgrounding, exit code 0).

Ersetzt PR #3. Ticket T-20260820-729932431, T-20260926-902661741.

* fix(lint): split combined import to satisfy ruff E401

---------

Co-authored-by: Lukas Geiger <lukas@um-bruch.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant