Feat/usenix support - #69
Open
chasecheese wants to merge 3 commits into
Open
Conversation
Add URL-to-URL regex pairs so the navigation button/hotkey switches between a USENIX conference presentation page and its paper PDF, in both directions, for all of NSDI/OSDI/ATC/FAST/Security/SOUPS and co-located workshops. A survey of a representative paper from every NSDI/OSDI/ATC/FAST/Security edition (2012-2026) shows two structural URL generations plus per-paper irregularities. The rules cover every transformation that is derivable from the URL alone: - Legacy technical-sessions form <-> /system/files/conference/<conf>/... subdir - Flat form (2018+) <-> /system/files/<conf>-<slug>.pdf - PDF -> presentation tolerates optional paper-/papers- infixes - USENIX Security sec<NN> file prefix mapped back to the usenixsecurity<NN> path Cases that are not derivable from the URL (pre-2014 submission-number filenames, per-paper slug truncation/underscore/_N suffixes, the 2018 flat-path/subdir-PDF transition) are left unsupported, consistent with the best-effort nature of the extension's other site mappings. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Cover the legacy -paper- format (NSDI'14, OSDI'14), the flat format across families and years (NSDI/OSDI/ATC/FAST/Security/SOUPS/WOOT, '18-'26), and the reverse-only wins (flat paper- infix NSDI'20, legacy papers- infix FAST'16, sec<NN> prefix Security'17/'20). All verified against live URLs. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
🚧 Files skipped from review as they are similar to previous changes (3)
WalkthroughUSENIX URL rewrite rules were added to both browser replacement tables. The PR also expands YAML testcase coverage for the new URL shapes and updates the README feature list to mention USENIX. ChangesUSENIX URL rewrite support
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@chrome/target_url_regexp_replace.js`:
- Around line 34-38: Add a USENIX Security-specific replacement before the
generic USENIX forward rules so first-match-wins routes `usenixsecurity<NN>`
presentation pages to the correct `sec<NN>` PDF filenames instead of the current
`usenixsecurity<NN>` prefix. Update the matching entries in
`chrome/target_url_regexp_replace.js` (the rule set around the USENIX conference
presentation mappings) so the rewritten URL uses the `sec` prefix for Security
papers while leaving the other conference patterns unchanged.
In `@firefox/target_url_regexp_replace.js`:
- Around line 34-38: Add a USENIX Security-specific rewrite rule before the
generic USENIX conference presentation rules in target_url_regexp_replace.js,
since Firefox uses the first match. Update the handling in the rule set around
the presentation-page mappings so usenixsecurity<NN> pages rewrite to
sec<NN>-...pdf instead of usenixsecurity<NN>-...pdf, and ensure the existing
generic conference rule remains below it for other conferences.
🪄 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: CHILL
Plan: Pro
Run ID: d3984a48-db86-4332-82dc-3926a24bc805
📒 Files selected for processing (4)
README.mdchrome/target_url_regexp_replace.jsfirefox/target_url_regexp_replace.jstests/testcases/testcases.yaml
USENIX Security paper PDFs use a sec<NN> file prefix (through 2022), not the
usenixsecurity<NN> conference path code, so the generic forward rules were
sending those presentation pages to non-existent PDF URLs (first-match-wins).
Add two Security-specific forward rules ahead of the generic ones:
- legacy technical-sessions (<=2017): -> conference/usenixsecurity<NN>/sec<NN>-<slug>.pdf
- flat 2019-2022: -> sec<NN>-<slug>.pdf (2023+ switched to the
usenixsecurity<NN> prefix and falls through)
Fixes e.g. sec17-alexopoulos and sec20-afek (forward 34->38 of the 64-paper
survey). The remaining Security forward misses (sec12-16/18/19) carry
non-derivable submission numbers, underscore/_N variants, or the 2018 subdir
transition and stay best-effort. Add both-direction Security testcases.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds USENIX conference papers support, matching the support already provided for arXiv, OpenReview, NIPS, PMLR, CVF, JMLR, IEEE, and ACL.
Navigates e.g. between:
https://www.usenix.org/conference/nsdi26/presentation/chen-jiayihttps://www.usenix.org/system/files/nsdi26-chen-jiayi.pdfScope
USENIX URLs have two structural generations plus a long tail of per-paper irregularities. LLM agents and me surveyed a representative paper from every NSDI/OSDI/ATC/FAST/USENIX Security edition 2012–2026, and added every transformation derivable from the URL. Measured coverage: presentation -> PDF 34/64, PDF -> presentation 47/64.
Not URL-derivable, hence unsupported (best-effort, like the other site mappings): pre-2014 submission-number filenames, per-paper slug truncation/underscore/
_Nsuffixes, and the 2018 flat-path/subdir-PDF transition.Tests
Added navigation testcases spanning the format/era matrix to
tests/testcases/testcases.yaml, all verified against live URLs.Summary by CodeRabbit
Summary by CodeRabbit
New Features
Tests
sec<NN>Security variants.