Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
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 |
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.
Courses exposed through the Task Center need a workflow separate from the chapter queue. This draft adds sequential group traversal and platform-state verification for videos, document reading, chapter synchronization, homework, topic discussions and AI practice. Unsupported thinking questions stay explicit, and Task Center failures remain isolated from chapter study.
The integration includes a discussion-board CLI, account-local configuration/cookies/cache, a setup wizard, question/answer parsing and durable review records. Substantive generated text is shown and recorded before requests; recording or validation failures block the affected submission. Cancelled, failed, unconfirmed and platform-accepted results remain distinct. Objective answers stay in runtime traces. Real duration requirements, request pacing and default-disabled quiz redo from #626 are preserved.
Scope and dependencies
This draft is the complete runnable integration. It includes prerequisites separately proposed in #637 (privacy/publication guards) and #638 (installable packaging); those smaller PRs are independently reviewable and should be considered first. Their overlap will need removal/rebasing after they merge. The remaining workflow modules share the CLI and local-data layer.
The branch starts from upstream main without importing the fork's development history. It preserves upstream app.py, license/author attribution, existing issue templates and compatibility exports. Public documentation follows the existing README; private handoffs, captures, account data and working notes are excluded.
Validation and limits
50ac130) passed Linux publication checks, all 366 offline tests and installed-wheel checks. The same commit passed Windows executable packaging, its help smoke test and artifact upload.Upstream Codacy now passes at
50ac130with zero added issues. CodeRabbit reports success for this draft, but draft status skips its full technical review.The quality refactor addresses the upstream Codacy findings through documentation normalization, smaller functions, removal of unused code and targeted security fixes. It does not broadly disable the upstream analysis rules.
Maintainer feedback on scope and API boundaries is welcome before promoting this draft for merge.