chore: add SECURITY.md, CODEOWNERS, and .editorconfig#6
Conversation
Objective: The repository lacks basic governance files for security reporting, automatic PR reviewer assignment, and editor consistency. Approach: Add three minimal governance files: - SECURITY.md under .github/ — private email channel for vulnerability reports, aligned with the upstream opendataloader-pdf policy. - CODEOWNERS under .github/ — auto-assigns reviewers: @suji-cho for docs, and @hyunhee-jo @bundolee @hnc-jglee for code. - .editorconfig at root — consistent indent (4 spaces), UTF-8, LF line endings, trailing whitespace handling. Heavier governance docs (CONTRIBUTING, CODE_OF_CONDUCT, SUPPORT) are intentionally omitted because this is a small integration package, not a main engine project; comparable packages (langchain-opendataloader-pdf, argilla-llama-index, toolbox-llamaindex) also skip them. Evidence: N/A — documentation and configuration files only, no behavior change. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds three repository files: Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/CODEOWNERS:
- Around line 2-5: The CODEOWNERS file currently lists the generic "*" pattern
after the "*.md" pattern causing last-match-wins to assign Markdown files to the
default owners; reorder the lines so that the "*" pattern appears first and the
"*.md `@suji-cho`" line comes after it (i.e., move the "*.md `@suji-cho`" entry
below the "* `@hyunhee-jo` `@bundolee` `@hnc-jglee`" entry) so "*.md" matches last and
assigns ownership to `@suji-cho`.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: ae558894-121e-465c-aadd-6b71e0f14ee8
📒 Files selected for processing (3)
.editorconfig.github/CODEOWNERS.github/SECURITY.md
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Objective
The repository lacks basic governance files that reviewers and contributors
expect: no channel for reporting security issues, no automatic PR reviewer
assignment, and no shared editor configuration. This creates friction for
anyone discovering the project via PyPI or the upstream LlamaIndex docs PR.
Approach
Add three minimal governance files, aligned with the upstream
opendataloader-pdfrepository:.github/SECURITY.md— private email channel(open.dataloader@hancom.com)
for vulnerability reports, with a 3 working day response SLA.
.github/CODEOWNERS— auto-assigns reviewers:@suji-chofor docs,@hyunhee-jo @bundolee @hnc-jgleefor code..editorconfig— shared indent (4 spaces), UTF-8, LF line endings,trailing whitespace handling consistent across editors.
Heavier governance docs (CONTRIBUTING, CODE_OF_CONDUCT, SUPPORT) are
intentionally omitted because this is a small integration package, not
a main engine project. Comparable external integrations
(
langchain-opendataloader-pdf,argilla-llama-index,toolbox-llamaindex)also skip them.
Evidence
Documentation and configuration files only — no runtime behavior change.
.github/(GitHub-recognized path)*.mdand*.editorconfigat root (auto-discovered by editors)Summary by CodeRabbit
Documentation
Chores