Skip to content

update summary to auto - #1

Merged
tibo-openai merged 1 commit into
mainfrom
dev/tcreech/auto-summary
Apr 16, 2025
Merged

tibo-openai merged 1 commit into
mainfrom
dev/tcreech/auto-summary

Conversation

@trevorcreech

Copy link
Copy Markdown
Contributor

No description provided.

@trevorcreech
trevorcreech requested a review from ibigio April 16, 2025 16:59
@tibo-openai
tibo-openai merged commit 443ffb7 into main Apr 16, 2025
@tibo-openai
tibo-openai deleted the dev/tcreech/auto-summary branch April 16, 2025 17:44
rumple pushed a commit to rumple/codex that referenced this pull request Apr 24, 2025
shepting pushed a commit to shepting/codex that referenced this pull request Sep 16, 2025
@mzltest mzltest mentioned this pull request Sep 24, 2025
DioNanos referenced this pull request in DioNanos/codex-termux Nov 15, 2025
Critical bugfix - auto-update detection now working

Fixes:
- Version parser now handles -termux suffix correctly
- Tag parser accepts both rust-v* and v*-termux formats
- Added test coverage for Termux tag validation

Technical details:
- v0.58.0 upstream merge overwrote -termux suffix support
- New upstream code rejected v*-termux tag format
- Prevented version.json creation and update notifications

Affected versions: v0.58.0 through v0.58.3

Termux patches (4 total):
- Patch #1: Browser login fix (termux-open-url)
- Patch #2: RAM optimizations (lto=false, codegen-units=16)
- Patch #3: Auto-update URL (@mmmbuto/codex-cli-termux)
- Patch #4: Auto-update detection (this release)
DioNanos referenced this pull request in DioNanos/codex-termux Nov 21, 2025
Upstream changes (40+ commits):
- New: codex-shell-tool-mcp MCP server
- New: execpolicycheck CLI command
- New: TUI animations toggle feature
- New: Shell timeout increased to 1 hour
- Refactor: execpolicy migration (execpolicy2 -> execpolicy)
- Removed: tiktoken-rs, shell_command feature
- Improved: FreeBSD portability, fuzzy search (8->20 results)

Termux patches verified compatible:
- #1 Browser login (termux-open-url) ✅
- #2 RAM optimizations (lto=false, codegen-units=16) ✅
- #8 Bash execution (sandbox, LD_*, shell detection) ✅
- #9 Auto-update execution ✅

Stats: 195 files changed, +5915 insertions, -2293 deletions

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
DioNanos referenced this pull request in DioNanos/codex-termux Nov 27, 2025
- 8 new tests to validate Termux patches after upstream merge
- Critical tests for Patch #1, #2, #4, #9 (conflicting patches)
- Test auto-update execution, browser login, RAM optimization
- Essential for post-merge validation before npm publish

Test suite v1.2 → v1.3 (90 total tests)
DioNanos referenced this pull request in DioNanos/codex-termux Nov 27, 2025
Merged 61 commits from openai/codex upstream (Nov 21-27, 2025).

✅ ALL TERMUX PATCHES PRESERVED:
- Patch #1: Browser login (termux-open-url) - VERIFIED ✅
- Patch #2: RAM optimization (lto=false) - VERIFIED ✅
- Patch #4: Auto-update URL (DioNanos/codex-termux) - VERIFIED ✅
- Patch #5: Version parser (-termux suffix) - VERIFIED ✅
- Patch #6: NPM package name (@mmmbuto/codex-cli-termux) - VERIFIED ✅
- Patch #9: Auto-update execution (npm install) - VERIFIED ✅

NEW UPSTREAM FEATURES:
- Enterprise skip upgrade config (check_for_update_on_startup)
- Unified exec pruning strategy
- Custom env for unified exec process
- Compaction events
- Thread/turn ID in notifications
- Config management app-server
- Cargo audit workflow + cargo-deny
- NetBSD build fixes
- Windows sandbox enhancements
- Documentation improvements

UPSTREAM CHANGES:
- 232 files changed
- +11247 insertions, -7808 deletions
- New codex-api/ and codex-client/ crates
- Rollout session error improvements
- Apply patch parsing fixes

CONFLICTS RESOLVED:
- README.md: Merged Testing section (Termux) + Documentation links (upstream)

NEXT: Compilation + Category 13 test suite validation
DioNanos referenced this pull request in DioNanos/codex-termux Jan 8, 2026
…8+ requirement

- Upstream: OpenAI Codex rust-v0.79.0
- Termux patches validated: #1-#6, #8, #9
- Testing: 47/49 passed (2 skipped: WebSearch, Git)
- Documentation:
  - Added GLM-4.7 quickstart guide (docs/GLM4.7-quickstart.md)
  - Updated README with GLM-4.7 Path 2 in Quickstart
  - Node.js requirement updated to >=18 (recommended >=22)
  - Web search safety warning added
  - Tagline updated to "Built from upstream"
- Package: npm-package synced with main README
- Binaries: codex (59M), codex-exec (34M), codex-tui (41M) verified
charley-oai added a commit that referenced this pull request Jan 9, 2026
charley-oai added a commit that referenced this pull request Jan 10, 2026
Agent wouldn't "see" attached images and would instead try to use the
view_file tool:
<img width="1516" height="504" alt="image"
src="https://github.com/user-attachments/assets/68a705bb-f962-4fc1-9087-e932a6859b12"
/>

In this PR, we wrap image content items in XML tags with the name of
each image (now just a numbered name like `[Image #1]`), so that the
model can understand inline image references (based on name). We also
put the image content items above the user message which the model seems
to prefer (maybe it's more used to definitions being before references).

We also tweak the view_file tool description which seemed to help a bit

Results on a simple eval set of images:

Before
<img width="980" height="310" alt="image"
src="https://github.com/user-attachments/assets/ba838651-2565-4684-a12e-81a36641bf86"
/>

After
<img width="918" height="322" alt="image"
src="https://github.com/user-attachments/assets/10a81951-7ee6-415e-a27e-e7a3fd0aee6f"
/>

```json
[
  {
    "id": "single_describe",
    "prompt": "Describe the attached image in one sentence.",
    "images": ["image_a.png"]
  },
  {
    "id": "single_color",
    "prompt": "What is the dominant color in the image? Answer with a single color word.",
    "images": ["image_b.png"]
  },
  {
    "id": "orientation_check",
    "prompt": "Is the image portrait or landscape? Answer in one sentence.",
    "images": ["image_c.png"]
  },
  {
    "id": "detail_request",
    "prompt": "Look closely at the image and call out any small details you notice.",
    "images": ["image_d.png"]
  },
  {
    "id": "two_images_compare",
    "prompt": "I attached two images. Are they the same or different? Briefly explain.",
    "images": ["image_a.png", "image_b.png"]
  },
  {
    "id": "two_images_captions",
    "prompt": "Provide a short caption for each image (Image 1, Image 2).",
    "images": ["image_c.png", "image_d.png"]
  },
  {
    "id": "multi_image_rank",
    "prompt": "Rank the attached images from most colorful to least colorful.",
    "images": ["image_a.png", "image_b.png", "image_c.png"]
  },
  {
    "id": "multi_image_choice",
    "prompt": "Which image looks more vibrant? Answer with 'Image 1' or 'Image 2'.",
    "images": ["image_b.png", "image_d.png"]
  }
]
```
DioNanos referenced this pull request in DioNanos/codex-termux Jan 10, 2026
- Updated workspace version to 0.80.0-termux
- All Termux patches verified functional (#1-#6, #9 active)
- Patch #8 no longer required (resolved upstream by PR openai#8951)
- Process hardening removed from Codex CLI improves bash execution
- Test suite: 49/49 passed (CODEX_TEST_REPORT_v0.80.0.md)
- Upstream changes:
  * Thread fork endpoints (conversation branching)
  * Requirements/list API
  * Elevated sandbox onboarding NUX
  * Skills explicit invocation via V2 API
  * Metrics capabilities (otel/metrics module)
  * Removed process hardening from Codex CLI
- Binaries compiled: codex (60M), codex-tui (42M), codex-exec (35M), codex-app-server (38M)
- Updated patches/README.md for v0.80.0
DioNanos referenced this pull request in DioNanos/codex-termux Jan 10, 2026
- Remove old test report (CODEX_TEST_REPORT_v0.79.0.md)
- Update version references from 0.79.0 to 0.80.0
- Update upstream reference from rust-v0.79.0 to rust-v0.80.0
- Update test date from 2026-01-08 to 2026-01-10
- Update test results to 49/49 passed (0 skipped)
- Update patch description: 8 patches (#1-#6, #9) - Patch #8 resolved upstream
- Sync npm-package/README.md with root README.md
charley-oai added a commit that referenced this pull request Jan 12, 2026
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.