Skip to content

Add PDF-aware preliminary investigation to arXiv research - #55644

Merged
pelikhan merged 4 commits into
mainfrom
copilot/add-pdf-aware-preliminary-investigator
Aug 25, 2026
Merged

Add PDF-aware preliminary investigation to arXiv research#55644
pelikhan merged 4 commits into
mainfrom
copilot/add-pdf-aware-preliminary-investigator

Conversation

Copilot AI commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

The daily arXiv workflow previously screened papers from abstracts only and redacted arXiv links in generated discussions. This adds a best-effort PDF evidence pass before screening.

  • PDF investigation
    • Imports the existing Kreuzberg MCP server.
    • Adds preliminary-investigator to download and extract paper PDFs.
    • Enriches screening and ranking with concise methodology/results notes.
    • Continues with abstract-only processing if retrieval or extraction fails.
{"title":"...","abstract":"...","preliminary_note":"Technique and results found in the PDF"}
  • Network and output policy

    • Allows arxiv.org and export.arxiv.org for PDF retrieval.
    • Allows arXiv URLs in safe-output discussions.
  • Kreuzberg shared component

    • Converts usage documentation from hidden HTML-comment content to normal Markdown.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Add PDF-aware preliminary investigation sub-agent Add PDF-aware preliminary investigation to arXiv research Aug 25, 2026
Copilot AI requested a review from pelikhan August 25, 2026 04:55
@github-actions

Copy link
Copy Markdown
Contributor

Hey @copilot-swe-agent 👋 — thanks for implementing the PDF-aware preliminary investigation enhancement to the daily arXiv researcher! This is a solid improvement that directly addresses the request in #55400.

What looks good:

  • The PDF investigation pass is well-integrated with graceful fallback if extraction fails — it won't break the pipeline.
  • Network policy is clearly defined (arxiv.org and export.arxiv.org allowed).
  • Kreuzberg MCP server integration leverages existing, vetted infrastructure.
  • The move of kreuzberg usage docs from HTML comments to Markdown is a security-positive change (blocks hidden-content scanner flags).
  • Clear, structured PR description with all implementation details.
  • Focused scope: three related files only.

This looks ready for review by maintainers. The changes are aligned with the project's agentic workflow patterns and directly implement a feature request from the core team discussion.

Generated by ✅ Contribution Check · copilot · auto · 56.2 AIC · ⌖ 6.18 AIC · ⊞ 9.3K ·

# Excluded feature-flag-gated operations:
# - "extract_structured" # Requires liter-llm feature flag at build time
---
<!--

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot keep the comments, and only expose a tiny amount of prompting.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed in f328a86: retained the component documentation as comments and reduced the exposed prompt to the extract_file instruction.

@pelikhan
pelikhan marked this pull request as ready for review August 25, 2026 05:06
Copilot AI balanced review requested due to automatic review settings August 25, 2026 05:06
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds PDF-based evidence gathering to the daily arXiv research workflow before screening and ranking.

Changes:

  • Adds a preliminary PDF investigator using Kreuzberg.
  • Allows arXiv network access and discussion links.
  • Exposes Kreuzberg usage documentation and updates generated output.
Show a summary per file
File Description
.github/workflows/daily-arxiv-researcher.md Adds PDF investigation and enriched screening.
.github/workflows/shared/mcp/kreuzberg.md Makes MCP documentation visible.
.github/workflows/daily-arxiv-researcher.lock.yml Updates compiled workflow metadata and network policy.

Review details

  • Files reviewed: 3/3 changed files
  • Comments generated: 4
  • Review effort level: Balanced

# Resolved workflow manifest:
# Imports:
# - shared/crush.md
# - shared/mcp/kreuzberg.md

## Step 1b: Investigate Paper PDFs

For each paper in `papers`, invoke the `preliminary-investigator` sub-agent with its full paper object. Merge the returned `pdf_fetched` and `preliminary_note` fields into that paper object.

For each paper in `papers`, invoke the `preliminary-investigator` sub-agent with its full paper object. Merge the returned `pdf_fetched` and `preliminary_note` fields into that paper object.

PDF extraction failures are non-fatal: continue screening and ranking with an empty `preliminary_note` when `pdf_fetched` is `false`.
Comment on lines +267 to +268
1. Download `https://arxiv.org/pdf/<id>` to `/tmp/gh-aw/agent/arxiv/<id>.pdf` with `curl`. If the download fails, return `{"pdf_fetched": false, "preliminary_note": ""}`.
2. Use the `kreuzberg` MCP server's `extract_bytes` tool with the PDF's base64 content to extract its text. If extraction fails or produces no text, return `{"pdf_fetched": false, "preliminary_note": ""}`.
Copilot AI requested a review from pelikhan August 25, 2026 05:18
@pelikhan
pelikhan merged commit c901870 into main Aug 25, 2026
@pelikhan
pelikhan deleted the copilot/add-pdf-aware-preliminary-investigator branch August 25, 2026 05:22
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.

[q] Add PDF-aware preliminary investigation sub-agent + allow arXiv URLs in reports

3 participants