Skip to content

fix(analyze): aggregate proof URLs across chunks and increase default timeouts#19

Open
tejasprasad2008-afk wants to merge 2 commits into
ni5arga:mainfrom
tejasprasad2008-afk:split/2-engine-fixes
Open

fix(analyze): aggregate proof URLs across chunks and increase default timeouts#19
tejasprasad2008-afk wants to merge 2 commits into
ni5arga:mainfrom
tejasprasad2008-afk:split/2-engine-fixes

Conversation

@tejasprasad2008-afk

Copy link
Copy Markdown

Description:

This PR addresses two critical stability issues in the analysis engine:

Changes:

  • Identity Proof Aggregation: Fixed a bug where publicProofUrls (LinkedIn, GitHub, etc.) were only being collected from the first chunk that identified a user. It now correctly
    aggregates all proof URLs found across the entire analyzed footprint.
  • Stable Timeouts: Increased internal timeouts to 180s (Primary) and 120s (Compressed). This prevents the tool from timing out and crashing when using slower LLM providers or
    processing dense chunks of history.

@tejasprasad2008-afk

Copy link
Copy Markdown
Author

hold on im not done yet

@tejasprasad2008-afk

Copy link
Copy Markdown
Author

@ni5arga can u run a copilot on this one to see if anything fails?

@ni5arga ni5arga requested a review from Copilot June 5, 2026 17:06
@ni5arga

ni5arga commented Jun 5, 2026

Copy link
Copy Markdown
Owner

alr sure

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@tejasprasad2008-afk

Copy link
Copy Markdown
Author

yaay

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread src/analyze.ts Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@tejasprasad2008-afk

Copy link
Copy Markdown
Author

ok that was just because the documentation of PR change intended a 120s timeout and the actual timeout was 117s .😅

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.

3 participants