Skip to content

PR-Build lässt jetzt Tests laufen#1

Merged
bsommerfeld merged 1 commit into
bsommerfeld:masterfrom
LongdeLao:codex/pr-build-run-unit-tests
Jun 30, 2026
Merged

PR-Build lässt jetzt Tests laufen#1
bsommerfeld merged 1 commit into
bsommerfeld:masterfrom
LongdeLao:codex/pr-build-run-unit-tests

Conversation

@LongdeLao

@LongdeLao LongdeLao commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Was?

PR-Build läuft jetzt erst durch ./mvnw clean test und baut danach wie gehabt die Test-Artefakte mit -DskipTests.

Warum?

Bisher war der PR-Build eher „kompiliert schon, passt schon“. Jetzt fliegen kaputte Unit-Tests früher auf, bevor sie bis zum Release durch-yoloen.

Check

YAML lokal geparst. Maven lokal nicht ausführbar, weil hier kein Java Runtime installiert ist.

@LongdeLao LongdeLao changed the title [codex] Run unit tests in PR build PR-Build lässt jetzt Tests laufen Jun 30, 2026
@bsommerfeld
bsommerfeld marked this pull request as ready for review June 30, 2026 19:23
@bsommerfeld

Copy link
Copy Markdown
Owner

Kompiliert schon, passt schon. (Tatsächlich nicht, aber das liegt an etwas anderem. Das nehme ich hier mit auf. Danke!)

@bsommerfeld
bsommerfeld merged commit 2052c56 into bsommerfeld:master Jun 30, 2026
1 check failed
bsommerfeld added a commit that referenced this pull request Jul 3, 2026
…hain across the story

The provenance gap closed by the line-scoped News tag had one remaining
leak: a fact debuts on a tagged line, the next line carries it forward
via the story memory as a paraphrase, and the sources launder away.

The model now REPORTS what it used - a step toward agentic self-
attribution using the field budget the slim-output era freed up:
- the brief's story memory numbers the shown prior headlines (#1..#3)
- the compose schema gains 'derivedFrom': an array of those ordinals
  (schema-forced, required; small integers on a short numbered list,
  which a 4B cites far more reliably than the long news uuids that
  killed the old sourceNewsIds field)
- inheritedRefs() maps cited ordinals back to the archived records and
  carries their newsRefs onto the new line, deduped by url, merged with
  the woven-in items; the News tag lights for either
- out-of-range ordinals (model mis-counts) are skipped, never fatal;
  no citation degrades to exactly the previous behavior

219 tests green (ordinal window mapping + url dedupe covered).
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.

2 participants