Skip to content

fix(llm): honor output ceiling in both passes - #75

Merged
ggrace519 merged 2 commits into
mainfrom
fix/max-output-token-override
Sep 5, 2026
Merged

fix(llm): honor output ceiling in both passes#75
ggrace519 merged 2 commits into
mainfrom
fix/max-output-token-override

Conversation

@ggrace519

Copy link
Copy Markdown
Owner

Summary

  • forward the resolved max_output_tokens ceiling to both pass-1 and pass-2 LLM requests
  • document that the CLI flag and environment setting apply per response without changing input admission
  • make synthetic provenance-test commits independent of global GPG-signing configuration

The pass-1 omission caused large repositories such as n8n to stop at the hard-coded 128000-token default even when --max-output-tokens 262000 or GRAPHLM_MAX_OUTPUT_TOKENS=262000 was supplied.

Closes #73
Closes #74

Verification

  • .venv/bin/pytest -q — 789 passed, 8 skipped, 2 collection warnings
  • .venv/bin/mypy graphlm --ignore-missing-imports — success across 31 source files
  • real n8n scan path with the LLM boundary instrumented — both requests received [262000, 262000] (previously [128000, 262000])
  • independent fresh-context review — APPROVE, no findings
  • git diff --check origin/main...HEAD — clean

Post-merge

Publish in the next patch release so installed 0.4.1 users can receive the fix with graphlm --upgrade.

🤖 Generated with Codex

ggrace519 and others added 2 commits September 4, 2026 21:48
Forward the resolved max-output-token ceiling to pass 1 as well as pass 2 so large directory trees can use the documented CLI or environment override. Keep input admission independent and document the per-pass behavior.

Co-Authored-By: Codex <noreply@openai.com>
Disable commit signing only for disposable repositories created by provenance tests so the offline suite does not depend on a users global Git config or GPG agent.

Co-Authored-By: Codex <noreply@openai.com>
@ggrace519
ggrace519 merged commit 4e51730 into main Sep 5, 2026
8 checks passed
@ggrace519
ggrace519 deleted the fix/max-output-token-override branch September 5, 2026 01:54
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.

test(git): isolate synthetic commits from global GPG signing fix(llm): honor max output token override in pass 1

1 participant