Repository navigation
Conversation
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 6 files
Shadow auto-approve: would not auto-approve because issues were found.
Re-trigger cubic
There was a problem hiding this comment.
0 issues found across 2 files (changes from recent commits).
Shadow auto-approve: would not auto-approve. Auto-approval blocked by 3 unresolved issues from previous reviews.
Re-trigger cubic
# Conflicts: # CHANGELOG.md
There was a problem hiding this comment.
0 issues found across 1 file (changes from recent commits).
Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.
Re-trigger cubic
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
No issues found across 7 files
Shadow auto-approve: would not auto-approve. This PR does not meet the repository auto-approval settings.
Re-trigger cubic
An Ubuntu package download in Unstructured-IO#4357's CI took 54 minutes for 262 MB, compared with 7–39 seconds on the other runners. The existing retry loop waits indefinitely for a command to return, so it cannot recover from a download that continues extremely slowly. This change shares dependency setup across the unit, extras, and ingest jobs. It applies 30-second HTTP/HTTPS inactivity timeouts, APT retries, and three bounded five-minute attempts for index updates and downloads. Packages are downloaded first and installed with `--no-download`, keeping dpkg installation outside the outer timeout. Setup appears as a separate step, and the PPA's implicit index refresh is replaced with an explicitly bounded refresh. The package set, Tesseract version check, and test commands are preserved. Persistently poor networking now produces a clear setup failure instead of an hour-long download; the change does not guarantee a faster mirror. Validation: Bash syntax, ShellCheck, shfmt, workflow YAML parsing, and whitespace checks passed. A mocked harness verified success, retry after timeout, exhaustion, and installation sequencing. A disposable Ubuntu 24.04 container validated the APT flags with strict index update, download-only fetching, and offline installation of libmagic-dev. The full helper also completed successfully in a disposable Ubuntu 24.04 ARM64 container, including LibreOffice, the Tesseract PPA, Korean OCR data, and diffstat; Tesseract reported 5.5.1. The repository release-version check passed on Ubuntu. GitHub CI passed on aa97013, including the unit, dependency-extras, ingest and Dockerfile jobs that use the new helper; no end-to-end CI speedup has been established.
This branch has not been deployed
1 participant
Summary
Replace worksheet-sized NetworkX graph construction with traversal over populated cells. Preserve four-neighbor connectivity, row-overlap merging, element ordering, text, HTML, and metadata. NetworkX remains a test dependency for comparison with the previous algorithm, and is removed from the XLSX runtime extra.
Compatibility
No new file-size, row, cell, or processing limit is introduced. Parsing, populated-cell semantics, and table rendering are unchanged. Exact serialized output matches the old algorithm for generated multi-sheet workbooks across header/HTML options and six existing XLSX fixtures. This is evidence of compatibility, not a guarantee for every possible workbook. Pandas still materializes worksheet data and a boolean mask; memory is reduced, not capped.
Validation
Synced with current main on October 1, 2026 (released as 0.27.15).
These measure component detection, not total API latency. Peak RSS is sampled every 5 ms, so it is a lower bound on the true peak.