Skip to content

fix(runartifact): shorten Windows reviewer paths - #576

Merged
rianjs merged 1 commit into
mainfrom
fix/575-shorten-windows-run-paths
Aug 27, 2026
Merged

fix(runartifact): shorten Windows reviewer paths#576
rianjs merged 1 commit into
mainfrom
fix/575-shorten-windows-run-paths

Conversation

@rianjs

@rianjs rianjs commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Use 12-character SHA components and a tuple hash for generated run artifact paths while preserving readable PR keys, full run UUIDs, and legacy resume paths.\n\nCloses #575
@rianjs

rianjs commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

No findings.

@rianjs-bot rianjs-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated PR Review

Reviewed commit: 3265aa17cc59
Profile: codex-rianjs-bot - Posting as: rianjs-bot[bot]

Summary

Reviewer Findings
go:implementation-tests 0

Reviewer Coverage

  • go:implementation-tests — complete (broad); skipped: none; constraints: none
Inspected files (3)
  • internal/pipeline/pipeline_test.go
  • internal/runartifact/runartifact.go
  • internal/runartifact/runartifact_test.go

0 PR discussion threads considered. 0 summarized; 0 resolved.


Completed in 2m 06s | gpt-5.6-terra | cr dev
Field Value
Model gpt-5.6-terra
Reviewers go:implementation-tests
Engine codex_cli · gpt-5.6-terra
Reviewed by cr · rianjs-bot[bot]
Duration 2m 06s wall · 1m 50s compute
Cost unavailable
Tokens 279.8k in / 2.6k out

Per-workstream usage

  • orchestrator-selection — gpt-5.6-terra
    • In: 17.2k
    • Out: 188
    • Cache read: 11.0k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 7s
  • go:implementation-tests — gpt-5.6-terra
    • In: 243.5k
    • Out: 2.3k
    • Cache read: 208.1k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 1m 38s
  • orchestrator-rollup — gpt-5.6-terra
    • In: 19.1k
    • Out: 39
    • Cache read: 16.1k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 4s

@rianjs
rianjs merged commit ab40daa into main Aug 27, 2026
10 checks passed
@rianjs
rianjs deleted the fix/575-shorten-windows-run-paths branch August 27, 2026 10:33
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