Repository navigation
Conversation
- Replace CSM-as-primary framing with accurate provider table; OMDb is the default and CSM API key is not publicly obtainable - Mark TMDB as 'Coming Soon' (raises NotImplementedError in factory.py) - Remove non-existent GitHub Pages documentation site links - Drop Enter keybinding (not present in any BINDINGS) - Remove 'Steam-inspired' marketing copy - Add Evidence Locker as an explicit feature bullet - Add F2 Interrogation Room shortcut and CSM manual-lookup workflow - Convert keybindings to a table matching actual app BINDINGS - Note clipboard image paste is macOS-only
Reviewer's GuideUpdates README.md to remove aspirational/incorrect documentation and align feature descriptions, keybindings, and provider details with the current implementation, including accurate API provider status, Evidence Locker behavior, CSM workflow, and storage/sync setup. Sequence diagram for manual Common Sense Media scoringsequenceDiagram
actor User
participant Room as Interrogation Room
participant Browser
participant CSM as commonsensemedia.org
participant Locker as Evidence Locker
User->>Room: Press F2
Room->>Browser: Open title search
Browser->>CSM: Search title
User->>Room: Enter category scores
Room->>Locker: Save manual dossier
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path="README.md" line_range="186" />
<code_context>
+| `l` | Open **The Lineup** (sequential review queue) |
+| `i` | Open **Interrogation Room** (manual data entry for selected title) |
+| `Space` | Flag selected title for follow-up |
+| `s` | Open **Preferences** (settings, weights, API provider & BYOS sync) |
+| `a` | Open **Advanced Options** (load CSV, run evaluation) |
+| `h` / `?` | Open **Help** overlay |
</code_context>
<issue_to_address>
**issue:** The keybindings table presents `s` as opening Preferences and `q` as quitting the application in every context, but the Lineup screen overrides `s` to skip the current title and `q` to leave the Lineup. Users following this table while in the Lineup perform different actions than documented.
**Triggers:** When a user presses `s` or `q` from The Lineup.
**Suggested fix:** Document the context-specific Lineup bindings, or qualify the table as applying only to the main application screen.
```suggestion
### ⌨️ Keybindings
*(The following bindings apply only to the main application screen; in The Lineup, `s` skips the current title and `q` leaves The Lineup.)*
```
</issue_to_address>Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: README.md:186
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| @@ -181,22 +185,23 @@ You can also configure sync directly in your environment or `~/.config/netflix-n | |||
|
|
|||
| ### ⌨️ Keybindings | |||
There was a problem hiding this comment.
issue: The keybindings table presents s as opening Preferences and q as quitting the application in every context, but the Lineup screen overrides s to skip the current title and q to leave the Lineup. Users following this table while in the Lineup perform different actions than documented.
Triggers: When a user presses s or q from The Lineup.
Suggested fix: Document the context-specific Lineup bindings, or qualify the table as applying only to the main application screen.
Suggested change
| ### ⌨️ Keybindings | |
| ### ⌨️ Keybindings | |
| *(The following bindings apply only to the main application screen; in The Lineup, `s` skips the current title and `q` leaves The Lineup.)* |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #34 +/- ##
=======================================
Coverage 87.27% 87.27%
=======================================
Files 25 25
Lines 2948 2948
=======================================
Hits 2573 2573
Misses 375 375 ☔ View full report in Codecov by Harness. |
This branch has not been deployed
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
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.
Summary
The README was written before much of the implementation was complete, leaving several inaccuracies based on aspirational features. This PR corrects it to reflect the app as it actually exists.
Changes
Removed (aspirational/inaccurate)
kilo59.github.io/netflix-narc/which is not a deployed GitHub Pages site; all doc links now point to localdocs/filesfactory.pyraisesNotImplementedErrorfor TMDB; it is now clearly labeled "Coming Soon"Enterkeybinding — not present in anyBINDINGSin the codebaseAdded (accurately reflects code)
F2shortcut in Interrogation Room — opens commonsensemedia.org in browser; key workflow for manually sourcing category scores without a CSM API keyBINDINGSinmain.pySummary by Sourcery
Update the README to accurately describe the application's current capabilities and usage.
Enhancements:
Documentation: