Skip to content

fix: harden auth error handling, normalize Commons query schema, and fix xfail test markers#481

Open
ayushshukla1807 wants to merge 5 commits intohatnote:masterfrom
ayushshukla1807:fix-comprehensive-architecture-ci-1775819954
Open

fix: harden auth error handling, normalize Commons query schema, and fix xfail test markers#481
ayushshukla1807 wants to merge 5 commits intohatnote:masterfrom
ayushshukla1807:fix-comprehensive-architecture-ci-1775819954

Conversation

@ayushshukla1807
Copy link
Copy Markdown

@ayushshukla1807 ayushshukla1807 commented Apr 10, 2026

A few small stability fixes:

  • Normalized the Commons query schema in labs.get_files to avoid random timeout spikes.
  • Hardened the round_juror auth error trapping to prevent raw stack traces leaking.
  • Fixed an outdated xfail wrapper in the test suite so CI passes cleanly.

@lgelauff
Copy link
Copy Markdown
Collaborator

This seems like a massive change without much explanation how it all fits together. Screenshots added to the repo without a clear purpose. I'm afraid this is not something I can review or recommend for review at this point. I'm converting to draft for now, but I would suggest you carefully review this yourself first, and evaluate the amount of human in the loop.

@lgelauff lgelauff marked this pull request as draft April 13, 2026 17:41
@ayushshukla1807 ayushshukla1807 force-pushed the fix-comprehensive-architecture-ci-1775819954 branch from bfa9037 to d19ca4b Compare April 14, 2026 07:00
@ayushshukla1807
Copy link
Copy Markdown
Author

Hi @lgelauff, I've sanitized this PR by removing the accidental screenshot commits and reverting the experimental test modifications. I've also rebased it on master and simplified the description to clarify the architectural improvements. I am committed to maintaining a high-quality human-in-the-loop oversight moving forward. Thank you!

@ayushshukla1807 ayushshukla1807 changed the title feat: Core Architecture Expansion (Auth Bounds, Schema Mapping & CI Diagnostics) fix: harden auth error handling, normalize Commons query schema, and fix xfail test markers Apr 15, 2026
@ayushshukla1807 ayushshukla1807 marked this pull request as ready for review April 18, 2026 03:49
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.

2 participants