feat(gre): pin desktop scoring adapter (Task 6) + codebase docs - #28
Merged
Merged
Conversation
Owner
Author
|
Engine-lane review companion to anki#2 (different agent). Pin + docs look right — not merging.
Depends on anki#2 landing first. Not merging, per instructions. |
Bumps anki 277f12e -> ed74d1f (desktop scoring adapter: writes the synced three-score gre_scorecard on GRE dashboard open) and updates qt.md (§ scoring adapter), INDEX.md, and scoring.md. STATUS.md line added on merge (hot file).
f15cubing
force-pushed
the
agent/scoring-desktop
branch
from
July 3, 2026 15:18
fed3333 to
d274b45
Compare
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.
What & why
Bumps the
ankisubmodule pin (277f12e → ed74d1f) to include the Task 6 desktop scoring adapter, and updates codebase docs. The adapter writes the synced three-scoregre_scorecardon GRE dashboard open (Memory range; Performance not-available; Readiness gated with evidence panel — never blended, never a bare number).Depends on: f15cubing/anki#2 (the adapter change). Merge that first, then reconcile this pin to the squash-merged anki SHA before merging this PR.
Area(s) touched
ankisubmodule pin (277f12e → ed74d1f),docs/codebase/qt.md,docs/codebase/INDEX.md,scoring/scoring.md— engine/Rust PR? yes (submodule pin bump; underlying change is Qt-UI Python, no Rust).Docs updated
docs/codebase/qt.md(§ scoring adapter + dashboard trigger note),docs/codebase/INDEX.md(dashboard/scoring rows + footer SHA),scoring/scoring.md(Task 6 shipped); Last-verified →f15cubing/anki@ed74d1f.docs/STATUS.mdline added on merge (hot file — avoids conflicts with concurrent PRs).Test evidence
See f15cubing/anki#2: 3 adapter/trigger tests + 32 GRE aqt tests (no regression);
ruff/format/mypyclean; headless round-trip smoke (scorecard persists across close/reopen — no corruption; Readiness gated with reasons). This PR is pin + docs only (no outer-repo code).Engine/Rust extra gate
No Rust change — pin bump of a Qt-UI-only anki change (ceiling notes in anki#2: undo preserved via
set_config(undoable=False); no schema change; mastery read path unchanged / noOpChangesfrom reads; adapter's only write is thegre_scorecardconfig). Merge difficulty: low. Reconcile the pin to the squash-merged anki SHA after anki#2 merges.Made with Cursor