Skip to content

Adding the documentation detection feature - #33

Open
Madhavi1108 wants to merge 8 commits into
Sushma-1706:mainfrom
Madhavi1108:documentation-detection
Open

Adding the documentation detection feature#33
Madhavi1108 wants to merge 8 commits into
Sushma-1706:mainfrom
Madhavi1108:documentation-detection

Conversation

@Madhavi1108

Copy link
Copy Markdown

This PR improves documentation detection and scoring by introducing a tree-based discovery approach instead of relying on a single hardcoded README path.

Closes #7

Type of Change

☑️ Enhancement
☑️ Backend
☑️ Documentation Analysis
☑️ Testing

Changes Made

  • Added tree-based documentation discovery using GraphQL repository trees
  • Added support for README variants with different casing and extensions (README.md, readme.md, README.rst, etc.)
  • Added support for documentation files located in docs/, doc/, and documentation/ directories
  • Implemented two-phase retrieval to fetch documentation content only after documentation files are detected
  • Added priority-based matching to select the most appropriate documentation file
  • Integrated updated documentation scoring into repository evaluation logic
  • Added consistent documentation detection behavior across both Python and TypeScript implementations
  • Added comprehensive test coverage with Vitest and Python unittest suites

How Has This Been Tested?

  • Added 12 Vitest test cases
  • Added 7 Python unittest test cases
  • Verified all 19 tests pass successfully
  • Verified TypeScript build completes successfully
  • Manually validated detection of README variants and documentation files in standard documentation directories

Checklist

☑️ My code follows the existing code style of the project
☑️ I have performed a self-review of my own code
☑️ I have added/updated tests where necessary
☑️ My changes do not introduce any new warnings or errors
☑️ I have linked the related issue

@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

@Madhavi1108 is attempting to deploy a commit to the sushma-1706's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Madhavi1108

Copy link
Copy Markdown
Author

Hi @Sushma-1706 ,
I have resolved the issue.
Kindly let me know if any changes are needed

@Madhavi1108

Copy link
Copy Markdown
Author

Hi @Sushma-1706 ,
I have resolved all merge conflicts so kindly let me know if any other changes are required

@Madhavi1108

Copy link
Copy Markdown
Author

Hi @Sushma-1706,
Thank you for your patience
The current pr is ready to merge

@Madhavi1108
Madhavi1108 force-pushed the documentation-detection branch from f35cfa8 to cbf5bec Compare June 7, 2026 03:43
@Madhavi1108

Copy link
Copy Markdown
Author

Hi @Sushma-1706 ,
I have added a detailed documentation for this branch as well.
Kindly let me know if any chnages are needed from my side

@Madhavi1108

Copy link
Copy Markdown
Author

Hi @Sushma-1706 ,
Just following up on this issue. I wanted to check if there are any changes or improvements needed from my side. Please let me know if there's anything I should update or address.
Thank you!

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.

Improve README detection beyond HEAD:README.md

1 participant