Skip to content

Add --upload-log support#129

Merged
konard merged 6 commits into
mainfrom
issue-128-9d904e7dd227
May 21, 2026
Merged

Add --upload-log support#129
konard merged 6 commits into
mainfrom
issue-128-9d904e7dd227

Conversation

@konard

@konard konard commented May 20, 2026

Copy link
Copy Markdown
Member

Fixes #128.

Summary

  • Adds --upload-log <id> to the JavaScript and Rust CLIs.
  • Resolves <id> through the existing execution store lookup, so UUIDs and isolation session names both work.
  • Validates the stored logPath, installs gh-upload-log with Bun or npm when it is missing, and runs gh-upload-log <logPath> with output streamed to the terminal.
  • Adds issue Add $ --upload-log support #128 case-study docs, online/repo research, requirements, root-cause notes, and release metadata.

Tests

  • cd js && bun run test
  • cd js && bun run check
  • cargo test --manifest-path rust/Cargo.toml
  • bun scripts/check-file-size.mjs
  • node scripts/check-test-parity.mjs

Notes

  • No screenshots are included because this is CLI behavior.

Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #128
@konard konard self-assigned this May 20, 2026
@konard konard changed the title [WIP] Add $ --upload-log support Add --upload-log support May 20, 2026
@konard konard marked this pull request as ready for review May 20, 2026 18:49
@konard

konard commented May 20, 2026

Copy link
Copy Markdown
Member Author

Working session summary

Implemented and pushed the fix to PR 129: #129

What changed:

  • Added --upload-log <id> support in both JS and Rust CLIs.
  • Resolves UUIDs/session names through the execution store and uploads the stored logPath.
  • Auto-installs gh-upload-log via Bun or npm when missing.
  • Streams uploader output to the terminal.
  • Added Windows-safe .cmd handling and direct PATH resolution.
  • Added tests, docs, case-study artifacts, and release metadata.

Validation:

  • Local JS tests/checks passed.
  • Local Rust tests passed.
  • Test parity passed.
  • GitHub Actions are green on latest commit c0f823c for both JavaScript CI/CD and Rust CI/CD.
  • PR is ready, working tree is clean, and main is merged into the branch.

This summary was automatically extracted from the AI working session output.

@konard

konard commented May 20, 2026

Copy link
Copy Markdown
Member Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.5
  • Provider: OpenAI
  • Public pricing estimate: $27.569858

📊 Context and tokens usage:

  • 481.5K / 1.1M (46%) input tokens, 59.7K / 128K (47%) output tokens

Total: (481.5K + 20.1M cached) input tokens, 59.7K output tokens, $27.569858 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Repository (83269KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard

konard commented May 20, 2026

Copy link
Copy Markdown
Member Author

✅ Ready to merge

This pull request is now ready to be merged:

  • All CI checks have passed
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

@konard konard merged commit a546dac into main May 21, 2026
25 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.

Add $ --upload-log support

1 participant