Correct shared agent-tooling contracts at the Workflows source

[stranske]

Why

The consumer-managed paths manifest in stranske/Workflows/renovate-presets/consumer-managed-paths.json
owns tools/llm_provider.py, tools/langchain_client.py, and the relevant scripts/langchain/*
files synced into TMP. Current source still overwrites explicit tracing preference, dynamically
imports re, permits label prefix false positives, always synthesizes the progress-review payload,
and silently clamps structured-output repair attempts. scripts/ci_cosmetic_repair.py also retains
naive UTC. Fixing only TMP would be overwritten by sync.

Scope

Correct these contracts in stranske/Workflows, add source tests, and propagate them through the
normal consumer sync path.

Non-Goals

Do not patch consumer copies directly or change provider/model selection policy. Do not increase LLM
repair retries without an explicit bounded policy. Scaffold-only completion does not count: source
changes without source tests and consumer-sync evidence fail this issue.

Tasks

• In tools/llm_provider.py, preserve an explicit LANGCHAIN_TRACING_V2 value instead of overwriting it when a LangSmith key exists; add cases to tests/tools/test_llm_provider.py.
• Replace inline __import__("re") in tools/langchain_client.py with the normal module import and retain reasoning-model tests in tests/tools/test_langchain_client.py.
• Replace bidirectional prefix matching in scripts/langchain/label_matcher.py with explicit token/alias rules and add doctor versus doc regression coverage.
• Make progress_reviewer.build_review_payload unconditional or add a real optional review field, and make structured_output.max_repair_attempts honor an explicit bounded policy rather than silently clamp to one.
• Replace naive UTC in scripts/ci_cosmetic_repair.py and add/adjust source-owned tests.
• Run gh workflow run maint-68-sync-consumer-repos.yml --repo stranske/Workflows, then use scripts/check_consumer_sync_drift.py to verify the exact source changes reach Trend_Model_Project without consumer-local edits.

Acceptance Criteria

• Workflows source tests for llm_provider, langchain_client, label matching, progress review, structured output, and cosmetic repair pass with named non-zero collections.
• An explicit LANGCHAIN_TRACING_V2=false remains false, doctor does not match doc, and requested repair-attempt policy is either honored or rejected explicitly rather than silently truncated.
• A generated consumer sync shows the relevant TMP files match Workflows source byte-for-byte.
• Deliberate-break gate: temporarily restore unconditional assignment of LANGCHAIN_TRACING_V2=true; the named source test must fail, then revert the edit.

Implementation Notes

File this in stranske/Workflows, not Trend_Model_Project. Use the repository's source-first sync
process and its own issue-format validator immediately before filing.

[github-actions]

This issue is not yet agent-processable. See docs/AGENT_ISSUE_FORMAT.md.

• Tasks must name a concrete file, symbol, path, config key, job, or command.

Advisory:

• 4 cited path(s) do not exist yet: stranske/Workflows/renovate-presets/consumer-managed-paths.json, stranske/Workflows, token/alias, add/adjust — expected when a task creates them; check for typos otherwise.

Dispatched the Agents Issue Optimizer format phase (attempt 1 of 3).

[github-actions]

This issue is not yet agent-processable. See docs/AGENT_ISSUE_FORMAT.md.

• Tasks must name a concrete file, symbol, path, config key, job, or command.

Advisory:

• 4 cited path(s) do not exist yet: stranske/Workflows/renovate-presets/consumer-managed-paths.json, stranske/Workflows, token/alias, add/adjust — expected when a task creates them; check for typos otherwise.

Dispatched the Agents Issue Optimizer format phase (attempt 2 of 3).

[stranske]

Automation-readiness repair: the only failing task now names the exact Maint 68 dispatch command and drift-verification script. The current body passes the source-owned issue-format validator locally, so it has been routed to auto-pilot instead of waiting for a third formatter retry.

[stranske]

Closer verifier disposition for merged PR #3083: current Workflows main contains UTC-safe cosmetic-repair handling and an unconditional progress-review payload; focused tests passed (25). However, the required source-to-consumer check against Trend_Model_Project currently reports consumer drift. The provider CONCERNS result is therefore not fully superseded. Reopened this source issue and requested a bounded verifier follow-up; next automation action is to reconcile the generated consumer sync and attach a clean byte-for-byte drift report before rerunning provider comparison.

[stranske]

Closer verifier follow-up (cursor): opened bounded evidence PR #3095 and pushed byte-aligned consumer-managed path copies to Trend_Model_Project@sync/workflows-delivery (6c9d80c4, updates sync PR #5819). Focused Workflows source tests: 134 passed. Next: merge Trend #5819 onto phase-3, then re-run verify:compare on merged #3083 before closing this issue.