Skip to content

itdove/lore#87: DocRepoIngester: avoid double file read in process_file - #97

Merged
itdove merged 1 commit into
mainfrom
87
Sep 8, 2026
Merged

itdove merged 1 commit into
mainfrom
87

Conversation

@itdove

@itdove itdove commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Description

#87: DocRepoIngester: avoid double file read in process_file

Related issue (if applicable)

Type of change

  • Bug fix
  • New feature
  • Security fix or detection change
  • Documentation or workflow change
  • Refactoring or maintenance

Security impact

Select exactly one of the following mutually exclusive options:

  • This change has no security impact.
  • This change affects security behavior; describe the threat, protection, and user-visible behavior below.

Testing

  • Tests added or updated where needed.
  • Related tests pass locally.
  • Manual verification completed, if applicable.

Commands run

# Record command evidence here

Checklist

  • The change is limited to this PR's purpose.
  • Documentation is updated where needed.
  • CHANGELOG.md is updated for notable changes, or no update is needed.
  • Backward compatibility was considered for the public SDK and configuration.
  • New dependencies have compatible licenses, if applicable.
  • UX contract tests are included when user-visible security behavior changes.

🤖 Generated with [Codex](https://github.com/openai/codex)

Co-Authored-By: Codex (gpt-5.6-luna) <noreply@openai.com>
@itdove
itdove marked this pull request as ready for review September 8, 2026 17:00
@itdove
itdove merged commit 9183c1c into main Sep 8, 2026
6 checks passed
@itdove
itdove deleted the 87 branch September 8, 2026 17:00
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