Skip to content

fix(assets): sample latest owner routes without blocking - #54

Merged
phodal merged 1 commit into
QoderAI:mainfrom
P0m32Kun:fix/asset-route-evidence
Aug 3, 2026
Merged

fix(assets): sample latest owner routes without blocking#54
phodal merged 1 commit into
QoderAI:mainfrom
P0m32Kun:fix/asset-route-evidence

Conversation

@P0m32Kun

@P0m32Kun P0m32Kun commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep the compact owner-route budget at 16 while selecting the newest observable filesystem mtimes
  • expose collection time, timestamp coverage, and per-route modifiedAt evidence
  • separate intentional route sampling from evidence-loss truncation so normal reviews do not fail on asset count alone
  • bound filesystem metadata probes to 32 concurrent operations

Traceability

Implements docs/specs/2026-08-03-asset-route-selection-freshness.md (AC-1 through AC-5).

Validation

  • npm run check on Node 24: 1020/1020 tests passed
  • npm and runtime package verification passed
  • focused doc-link graph tests passed 6/6

Risk

Filesystem mtimes can be absent or coarse. The envelope reports timestamped and untimestamped counts and uses deterministic fallback ordering instead of inferring freshness.

Implements docs/specs/2026-08-03-asset-route-selection-freshness.md. The compact baseline keeps its 16-route budget, records observable freshness, and separates intentional sampling from evidence-loss truncation. Validated with npm run check on Node 24 (1020 tests plus npm and runtime package verification).

Co-authored-by: Codex (GPT 5.6 Sol) <codex@openai.com>
@phodal
phodal merged commit 2df440c into QoderAI:main Aug 3, 2026
4 checks passed
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