Extend launcher to other platforms + more

- Switch to Qt 6.8 for repo default, as 6.7 depends on an older
libwebp/tiff which is unavailable on newer installs
- Drop tools/mac-x86, as we no longer need to test against Qt 5
- Add flags to cross compile wheels on Mac and Linux
- Bump glibc target to 2_36, building on Debian Stable
- Increase mpv timeout on macOS to allow for initial gatekeeper checks
- Ship both arm64 and amd64 uv on Linux, with a bash stub to pick
the appropriate arch.

.cargo/config.toml

@@ -10,6 +10,3 @@ PYTHONDONTWRITEBYTECODE = "1" # prevent junk files on Windows
 
 [term]
 color = "always"
-
-[target.'cfg(all(target_env = "msvc", target_os = "windows"))']
-rustflags = ["-C", "target-feature=+crt-static"]

.deny.toml

@@ -5,6 +5,9 @@
 db-path = "~/.cargo/advisory-db"
 db-urls = ["https://github.com/rustsec/advisory-db"]
 ignore = [
+  # pyoxidizer is stuck on an old ring version
+  "RUSTSEC-2025-0009",
+  "RUSTSEC-2025-0010",
   # burn depends on an unmaintained package 'paste'
   "RUSTSEC-2024-0436",
 ]
@@ -14,11 +17,12 @@ allow = [
   "MIT",
   "Apache-2.0",
   "Apache-2.0 WITH LLVM-exception",
-  "CDLA-Permissive-2.0",
   "ISC",
   "MPL-2.0",
+  "Unicode-DFS-2016",
   "BSD-2-Clause",
   "BSD-3-Clause",
+  "OpenSSL",
   "CC0-1.0",
   "Unlicense",
   "Zlib",

.gitignore

@@ -18,5 +18,3 @@ node_modules
 yarn-error.log
 ts/.svelte-kit
 .yarn
-.claude/settings.local.json
-.claude/user.md

.isort.cfg

@@ -0,0 +1,4 @@
+[settings]
+py_version=39
+known_first_party=anki,aqt,tests
+profile=black

.pylintrc

@@ -0,0 +1,48 @@
+[MASTER]
+ignore-patterns=.*_pb2.*
+persistent = no
+extension-pkg-whitelist=orjson,PyQt6
+init-hook="import sys; sys.path.extend(['pylib/anki/_vendor', 'out/qt'])"
+
+[REPORTS]
+output-format=colorized
+
+[MESSAGES CONTROL]
+disable=
+  R,
+  line-too-long,
+  too-many-lines,
+  missing-function-docstring,
+  missing-module-docstring,
+  missing-class-docstring,
+  import-outside-toplevel,
+  wrong-import-position,
+  wrong-import-order,
+  fixme,
+  unused-wildcard-import,
+  attribute-defined-outside-init,
+  redefined-builtin,
+  wildcard-import,
+  broad-except,
+  bare-except,
+  unused-argument,
+  unused-variable,
+  redefined-outer-name,
+  global-statement,
+  protected-access,
+  arguments-differ,
+  arguments-renamed,
+  consider-using-f-string,
+  invalid-name,
+  broad-exception-raised
+
+[BASIC]
+good-names =
+  id,
+  tr,
+  db,
+  ok,
+  ip,
+
+[IMPORTS]
+ignored-modules = anki.*_pb2, anki.sync_pb2, win32file,pywintypes,socket,win32pipe,pyaudio,anki.scheduler_pb2,anki.notetypes_pb2

.python-version

@@ -1 +1 @@
-3.13.5
+3.9.23

.ruff.toml

@@ -1,91 +1,2 @@
-lint.select = [
+target-version = "py39"
-  "E", # pycodestyle errors
+extend-exclude = []
-  "F", # Pyflakes errors
-  "PL", # Pylint rules
-  "I", # Isort rules
-  "ARG",
-  # "UP", # pyupgrade
-  # "B", # flake8-bugbear
-  # "SIM", # flake8-simplify
-]
-
-extend-exclude = ["*_pb2.py", "*_pb2.pyi"]
-
-lint.ignore = [
-  # Docstring rules (missing-*-docstring in pylint)
-  "D100", # Missing docstring in public module
-  "D101", # Missing docstring in public class
-  "D103", # Missing docstring in public function
-
-  # Import rules (wrong-import-* in pylint)
-  "E402", # Module level import not at top of file
-  "E501", # Line too long
-
-  # pycodestyle rules
-  "E741", # ambiguous-variable-name
-
-  # Comment rules (fixme in pylint)
-  "FIX002", # Line contains TODO
-
-  # Pyflakes rules
-  "F402", # import-shadowed-by-loop-var
-  "F403", # undefined-local-with-import-star
-  "F405", # undefined-local-with-import-star-usage
-
-  # Naming rules (invalid-name in pylint)
-  "N801", # Class name should use CapWords convention
-  "N802", # Function name should be lowercase
-  "N803", # Argument name should be lowercase
-  "N806", # Variable in function should be lowercase
-  "N811", # Constant imported as non-constant
-  "N812", # Lowercase imported as non-lowercase
-  "N813", # Camelcase imported as lowercase
-  "N814", # Camelcase imported as constant
-  "N815", # Variable in class scope should not be mixedCase
-  "N816", # Variable in global scope should not be mixedCase
-  "N817", # CamelCase imported as acronym
-  "N818", # Error suffix in exception names
-
-  # Pylint rules
-  "PLW0603", # global-statement
-  "PLW2901", # redefined-loop-name
-  "PLC0415", # import-outside-top-level
-  "PLR2004", # magic-value-comparison
-
-  # Exception handling (broad-except, bare-except in pylint)
-  "BLE001", # Do not catch blind exception
-
-  # Argument rules (unused-argument in pylint)
-  "ARG001", # Unused function argument
-  "ARG002", # Unused method argument
-  "ARG005", # Unused lambda argument
-
-  # Access rules (protected-access in pylint)
-  "SLF001", # Private member accessed
-
-  # String formatting (consider-using-f-string in pylint)
-  "UP032", # Use f-string instead of format call
-
-  # Exception rules (broad-exception-raised in pylint)
-  "TRY301", # Abstract raise to an inner function
-
-  # Builtin shadowing (redefined-builtin in pylint)
-  "A001", # Variable shadows a Python builtin
-  "A002", # Argument shadows a Python builtin
-  "A003", # Class attribute shadows a Python builtin
-]
-
-[lint.per-file-ignores]
-"**/anki/*_pb2.py" = ["ALL"]
-
-[lint.pep8-naming]
-ignore-names = ["id", "tr", "db", "ok", "ip"]
-
-[lint.pylint]
-max-args = 12
-max-returns = 10
-max-branches = 35
-max-statements = 125
-
-[lint.isort]
-known-first-party = ["anki", "aqt", "tests"]

.version

@@ -1 +1 @@
-25.09.2
+25.06

.vscode.dist/extensions.json

@@ -2,7 +2,7 @@
     "recommendations": [
         "dprint.dprint",
         "ms-python.python",
-        "charliermarsh.ruff",
+        "ms-python.black-formatter",
         "rust-lang.rust-analyzer",
         "svelte.svelte-vscode",
         "zxh404.vscode-proto3",

.vscode.dist/settings.json

@@ -18,7 +18,7 @@
         "out/qt",
         "qt"
     ],
-    "python.formatting.provider": "charliermarsh.ruff",
+    "python.formatting.provider": "black",
     "python.linting.mypyEnabled": false,
     "python.analysis.diagnosticSeverityOverrides": {
         "reportMissingModuleSource": "none"

.yarnrc.yml

@@ -1,2 +1 @@
 nodeLinker: node-modules
-enableScripts: false

CLAUDE.md

@@ -1,86 +0,0 @@
-# Claude Code Configuration
-
-## Project Overview
-
-Anki is a spaced repetition flashcard program with a multi-layered architecture. Main components:
-
-- Web frontend: Svelte/TypeScript in ts/
-- PyQt GUI, which embeds the web components in aqt/
-- Python library which wraps our rust Layer (pylib/, with Rust module in pylib/rsbridge)
-- Core Rust layer in rslib/
-- Protobuf definitions in proto/ that are used by the different layers to
-  talk to each other.
-
-## Building/checking
-
-./check (check.bat) will format the code and run the main build & checks.
-Please do this as a final step before marking a task as completed.
-
-## Quick iteration
-
-During development, you can build/check subsections of our code:
-
-- Rust: 'cargo check'
-- Python: './tools/dmypy', and if wheel-related, './ninja wheels'
-- TypeScript/Svelte: './ninja check:svelte'
-
-Be mindful that some changes (such as modifications to .proto files) may
-need a full build with './check' first.
-
-## Build tooling
-
-'./check' and './ninja' invoke our build system, which is implemented in build/. It takes care of downloading required deps and invoking our build
-steps.
-
-## Translations
-
-ftl/ contains our Fluent translation files. We have scripts in rslib/i18n
-to auto-generate an API for Rust, TypeScript and Python so that our code can
-access the translations in a type-safe manner. Changes should be made to
-ftl/core or ftl/qt. Except for features specific to our Qt interface, prefer
-the core module. When adding new strings, confirm the appropriate ftl file
-first, and try to match the existing style.
-
-## Protobuf and IPC
-
-Our build scripts use the .proto files to define our Rust library's
-non-Rust API. pylib/rsbridge exposes that API, and _backend.py exposes
-snake_case methods for each protobuf RPC that call into the API.
-Similar tooling creates a @generated/backend TypeScript module for
-communicating with the Rust backend (which happens over POST requests).
-
-## Fixing errors
-
-When dealing with build errors or failing tests, invoke 'check' or one
-of the quick iteration commands regularly. This helps verify your changes
-are correct. To locate other instances of a problem, run the check again -
-don't attempt to grep the codebase.
-
-## Ignores
-
-The files in out/ are auto-generated. Mostly you should ignore that folder,
-though you may sometimes find it useful to view out/{pylib/anki,qt/_aqt,ts/lib/generated} when dealing with cross-language communication or our other generated sourcecode.
-
-## Launcher/installer
-
-The code for our launcher is in qt/launcher, with separate code for each
-platform.
-
-## Rust dependencies
-
-Prefer adding to the root workspace, and using dep.workspace = true in the individual Rust project.
-
-## Rust utilities
-
-rslib/{process,io} contain some helpers for file and process operations,
-which provide better error messages/context and some ergonomics. Use them
-when possible.
-
-## Rust error handling
-
-in rslib, use error/mod.rs's AnkiError/Result and snafu. In our other Rust modules, prefer anyhow + additional context where appropriate. Unwrapping
-in build scripts/tests is fine.
-
-## Individual preferences
-
-See @.claude/user.md

CONTRIBUTORS

@@ -49,7 +49,6 @@ Sander Santema <github.com/sandersantema/>
 Thomas Brownback <https://github.com/brownbat/>
 Andrew Gaul <andrew@gaul.org>
 kenden
-Emil Hamrin <github.com/e-hamrin>
 Nickolay Yudin <kelciour@gmail.com>
 neitrinoweb <github.com/neitrinoweb/>
 Andreas Reis <github.com/nwwt>
@@ -64,7 +63,6 @@ Jakub Kaczmarzyk <jakub.kaczmarzyk@gmail.com>
 Akshara Balachandra <akshara.bala.18@gmail.com>
 lukkea <github.com/lukkea/>
 David Allison <davidallisongithub@gmail.com>
-David Allison <62114487+david-allison@users.noreply.github.com>
 Tsung-Han Yu <johan456789@gmail.com>
 Piotr Kubowicz <piotr.kubowicz@gmail.com>
 RumovZ <gp5glkw78@relay.firefox.com>
@@ -234,16 +232,6 @@ Spiritual Father <https://github.com/spiritualfather>
 Emmanuel Ferdman <https://github.com/emmanuel-ferdman>
 Sunong2008 <https://github.com/Sunrongguo2008>
 Marvin Kopf <marvinkopf@outlook.com>
-Kevin Nakamura <grinkers@grinkers.net>
-Bradley Szoke <bradleyszoke@gmail.com>
-jcznk <https://github.com/jcznk>
-Thomas Rixen <thomas.rixen@student.uclouvain.be>
-Siyuan Mattuwu Yan <syan4@ualberta.ca>
-Lee Doughty <32392044+leedoughty@users.noreply.github.com>
-memchr <memchr@proton.me>
-Max Romanowski <maxr777@proton.me>
-Aldlss <ayaldlss@gmail.com>
-
 ********************
 
 The text of the 3 clause BSD license follows:

Cargo.toml

@@ -33,8 +33,9 @@ git = "https://github.com/ankitects/linkcheck.git"
 rev = "184b2ca50ed39ca43da13f0b830a463861adb9ca"
 
 [workspace.dependencies.fsrs]
-version = "5.1.0"
+version = "4.0.0"
 # git = "https://github.com/open-spaced-repetition/fsrs-rs.git"
+# rev = "a7f7efc10f0a26b14ee348cc7402155685f2a24f"
 # path = "../open-spaced-repetition/fsrs-rs"
 
 [workspace.dependencies]
@@ -51,101 +52,103 @@ ninja_gen = { "path" = "build/ninja_gen" }
 unicase = "=2.6.0" # any changes could invalidate sqlite indexes
 
 # normal
-ammonia = "4.1.2"
+ammonia = "4.0.0"
-anyhow = "1.0.98"
+anyhow = "1.0.90"
-async-compression = { version = "0.4.24", features = ["zstd", "tokio"] }
+apple-bundles = "0.17.0"
+async-compression = { version = "0.4.17", features = ["zstd", "tokio"] }
 async-stream = "0.3.6"
-async-trait = "0.1.88"
+async-trait = "0.1.83"
-axum = { version = "0.8.4", features = ["multipart", "macros"] }
+axum = { version = "0.7", features = ["multipart", "macros"] }
-axum-client-ip = "1.1.3"
+axum-client-ip = "0.6"
-axum-extra = { version = "0.10.1", features = ["typed-header"] }
+axum-extra = { version = "0.9.4", features = ["typed-header"] }
-bitflags = "2.9.1"
+blake3 = "1.5.4"
-blake3 = "1.8.2"
+bytes = "1.7.2"
-bytes = "1.10.1"
+camino = "1.1.9"
-camino = "1.1.10"
+chrono = { version = "0.4.38", default-features = false, features = ["std", "clock"] }
-chrono = { version = "0.4.41", default-features = false, features = ["std", "clock"] }
+clap = { version = "4.5.20", features = ["derive"] }
-clap = { version = "4.5.40", features = ["derive"] }
+coarsetime = "0.1.34"
-coarsetime = "0.1.36"
+convert_case = "0.6.0"
-convert_case = "0.8.0"
+criterion = { version = "0.5.1" }
-criterion = { version = "0.6.0" }
+csv = "1.3.0"
-csv = "1.3.1"
+data-encoding = "2.6.0"
-data-encoding = "2.9.0"
 difflib = "0.4.0"
-dirs = "6.0.0"
+dirs = "5.0.1"
 dunce = "1.0.5"
-embed-resource = "3.0.4"
+embed-resource = "2.4"
 envy = "0.4.2"
-flate2 = "1.1.2"
+flate2 = "1.0.34"
-fluent = "0.17.0"
+fluent = "0.16.1"
-fluent-bundle = "0.16.0"
+fluent-bundle = "0.15.3"
-fluent-syntax = "0.12.0"
+fluent-syntax = "0.11.1"
 fnv = "1.0.7"
 futures = "0.3.31"
-globset = "0.4.16"
+glob = "0.3.1"
+globset = "0.4.15"
 hex = "0.4.3"
 htmlescape = "0.3.1"
 hyper = "1"
 id_tree = "1.8.0"
 inflections = "1.1.1"
-intl-memoizer = "0.5.3"
+intl-memoizer = "0.5.2"
-itertools = "0.14.0"
+itertools = "0.13.0"
 junction = "1.2.0"
+lazy_static = "1.5.0"
 libc = "0.2"
 libc-stdhandle = "0.1"
 maplit = "1.0.2"
-nom = "8.0.0"
+nom = "7.1.3"
 num-format = "0.4.4"
-num_cpus = "1.17.0"
+num_cpus = "1.16.0"
 num_enum = "0.7.3"
-once_cell = "1.21.3"
+once_cell = "1.20.2"
 pbkdf2 = { version = "0.12", features = ["simple"] }
-phf = { version = "0.11.3", features = ["macros"] }
+phf = { version = "0.11.2", features = ["macros"] }
-pin-project = "1.1.10"
+pin-project = "1.1.6"
-prettyplease = "0.2.34"
+plist = "1.7.0"
+prettyplease = "0.2.24"
 prost = "0.13"
 prost-build = "0.13"
-prost-reflect = "0.14.7"
+prost-reflect = "0.14"
 prost-types = "0.13"
-pulldown-cmark = "0.13.0"
+pulldown-cmark = "0.9.6"
-pyo3 = { version = "0.25.1", features = ["extension-module", "abi3", "abi3-py39"] }
+pyo3 = { version = "0.24", features = ["extension-module", "abi3", "abi3-py39"] }
-rand = "0.9.1"
+rand = "0.8.5"
-rayon = "1.10.0"
+regex = "1.11.0"
-regex = "1.11.1"
+reqwest = { version = "0.12.8", default-features = false, features = ["json", "socks", "stream", "multipart"] }
-reqwest = { version = "0.12.20", default-features = false, features = ["json", "socks", "stream", "multipart"] }
+rusqlite = { version = "0.30.0", features = ["trace", "functions", "collation", "bundled"] }
-rusqlite = { version = "0.36.0", features = ["trace", "functions", "collation", "bundled"] }
 rustls-pemfile = "2.2.0"
 scopeguard = "1.2.0"
-serde = { version = "1.0.219", features = ["derive"] }
+serde = { version = "1.0.210", features = ["derive"] }
-serde-aux = "4.7.0"
+serde-aux = "4.5.0"
-serde_json = "1.0.140"
+serde_json = "1.0.132"
-serde_repr = "0.1.20"
+serde_repr = "0.1.19"
-serde_tuple = "1.1.0"
+serde_tuple = "0.5.0"
 sha1 = "0.10.6"
-sha2 = { version = "0.10.9" }
+sha2 = { version = "0.10.8" }
+simple-file-manifest = "0.11.0"
 snafu = { version = "0.8.6", features = ["rust_1_61"] }
-strum = { version = "0.27.1", features = ["derive"] }
+strum = { version = "0.26.3", features = ["derive"] }
-syn = { version = "2.0.103", features = ["parsing", "printing"] }
+syn = { version = "2.0.82", features = ["parsing", "printing"] }
-tar = "0.4.44"
+tar = "0.4.42"
-tempfile = "3.20.0"
+tempfile = "3.13.0"
 termcolor = "1.4.1"
-tokio = { version = "1.45", features = ["fs", "rt-multi-thread", "macros", "signal"] }
+tokio = { version = "1.40", features = ["fs", "rt-multi-thread", "macros", "signal"] }
-tokio-util = { version = "0.7.15", features = ["io"] }
+tokio-util = { version = "0.7.12", features = ["io"] }
-tower-http = { version = "0.6.6", features = ["trace"] }
+tower-http = { version = "0.5", features = ["trace"] }
-tracing = { version = "0.1.41", features = ["max_level_trace", "release_max_level_debug"] }
+tracing = { version = "0.1.40", features = ["max_level_trace", "release_max_level_debug"] }
 tracing-appender = "0.2.3"
-tracing-subscriber = { version = "0.3.20", features = ["fmt", "env-filter"] }
+tracing-subscriber = { version = "0.3.18", features = ["fmt", "env-filter"] }
-unic-langid = { version = "0.9.6", features = ["macros"] }
+tugger-windows-codesign = "0.10.0"
+unic-langid = { version = "0.9.5", features = ["macros"] }
 unic-ucd-category = "0.9.0"
 unicode-normalization = "0.1.24"
 walkdir = "2.5.0"
-which = "8.0.0"
+which = "5.0.0"
-widestring = "1.1.0"
+winapi = { version = "0.3", features = ["wincon"] }
-winapi = { version = "0.3", features = ["wincon", "winreg"] }
+wiremock = "0.6.2"
-windows = { version = "0.61.3", features = ["Media_SpeechSynthesis", "Media_Core", "Foundation_Collections", "Storage_Streams", "Win32_System_Console", "Win32_System_Registry", "Win32_System_SystemInformation", "Win32_Foundation", "Win32_UI_Shell", "Wdk_System_SystemServices"] }
-wiremock = "0.6.3"
 xz2 = "0.1.7"
-zip = { version = "4.1.0", default-features = false, features = ["deflate", "time"] }
+zip = { version = "0.6.6", default-features = false, features = ["deflate", "time"] }
-zstd = { version = "0.13.3", features = ["zstdmt"] }
+zstd = { version = "0.13.2", features = ["zstdmt"] }
 
 # Apply mild optimizations to our dependencies in dev mode, which among other things
 # improves sha2 performance by about 21x. Opt 1 chosen due to

LICENSE

@@ -6,6 +6,8 @@ The following included source code items use a license other than AGPL3:
 
 In the pylib folder:
 
+ * The SuperMemo importer: GPL3 and 0BSD.
+ * The Pauker importer: BSD-3.
  * statsbg.py: CC BY 4.0.
 
 In the qt folder:

README.md

@@ -1,4 +1,4 @@
-# Anki®
+# Anki
 
 [![Build status](https://badge.buildkite.com/c9edf020a4aec976f9835e54751cc5409d843adbb66d043bd3.svg?branch=main)](https://buildkite.com/ankitects/anki-ci)
 

build/configure/src/aqt.rs

@@ -27,6 +27,7 @@ pub fn build_and_check_aqt(build: &mut Build) -> Result<()> {
     build_forms(build)?;
     build_generated_sources(build)?;
     build_data_folder(build)?;
+    build_macos_helper(build)?;
     build_wheel(build)?;
     check_python(build)?;
     Ok(())
@@ -38,6 +39,7 @@ fn build_forms(build: &mut Build) -> Result<()> {
     let mut py_files = vec![];
     for path in ui_files.resolve() {
         let outpath = outdir.join(path.file_name().unwrap()).into_string();
+        py_files.push(outpath.replace(".ui", "_qt5.py"));
         py_files.push(outpath.replace(".ui", "_qt6.py"));
     }
     build.add_action(
@@ -335,6 +337,27 @@ impl BuildAction for BuildThemedIcon<'_> {
     }
 }
 
+fn build_macos_helper(build: &mut Build) -> Result<()> {
+    if cfg!(target_os = "macos") {
+        build.add_action(
+            "qt:aqt:data:lib:libankihelper",
+            RunCommand {
+                command: ":pyenv:bin",
+                args: "$script $out $in",
+                inputs: hashmap! {
+                    "script" => inputs!["qt/mac/helper_build.py"],
+                    "in" => inputs![glob!["qt/mac/*.swift"]],
+                    "" => inputs!["out/env"],
+                },
+                outputs: hashmap! {
+                    "out" => vec!["qt/_aqt/data/lib/libankihelper.dylib"],
+                },
+            },
+        )?;
+    }
+    Ok(())
+}
+
 fn build_wheel(build: &mut Build) -> Result<()> {
     build.add_action(
         "wheels:aqt",
@@ -342,12 +365,7 @@ fn build_wheel(build: &mut Build) -> Result<()> {
             name: "aqt",
             version: anki_version(),
             platform: None,
-            deps: inputs![
+            deps: inputs![":qt:aqt", glob!("qt/aqt/**"), "qt/pyproject.toml"],
-                ":qt:aqt",
-                glob!("qt/aqt/**"),
-                "qt/pyproject.toml",
-                "qt/hatch_build.py"
-            ],
         },
     )
 }

build/configure/src/pylib.rs

@@ -68,8 +68,7 @@ pub fn build_pylib(build: &mut Build) -> Result<()> {
             deps: inputs![
                 ":pylib:anki",
                 glob!("pylib/anki/**"),
-                "pylib/pyproject.toml",
+                "pylib/pyproject.toml"
-                "pylib/hatch_build.py"
             ],
         },
     )?;

build/configure/src/python.rs

@@ -1,61 +1,34 @@
 // Copyright: Ankitects Pty Ltd and contributors
 // License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+use std::env;
+
 use anyhow::Result;
 use ninja_gen::action::BuildAction;
 use ninja_gen::archives::Platform;
 use ninja_gen::build::FilesHandle;
+use ninja_gen::command::RunCommand;
 use ninja_gen::copy::CopyFiles;
 use ninja_gen::glob;
+use ninja_gen::hashmap;
 use ninja_gen::input::BuildInput;
 use ninja_gen::inputs;
 use ninja_gen::python::python_format;
 use ninja_gen::python::PythonEnvironment;
+use ninja_gen::python::PythonLint;
 use ninja_gen::python::PythonTypecheck;
-use ninja_gen::python::RuffCheck;
+use ninja_gen::rsync::RsyncFiles;
 use ninja_gen::Build;
 
-/// Normalize version string by removing leading zeros from numeric parts
-/// while preserving pre-release markers (b1, rc2, a3, etc.)
-fn normalize_version(version: &str) -> String {
-    version
-        .split('.')
-        .map(|part| {
-            // Check if the part contains only digits
-            if part.chars().all(|c| c.is_ascii_digit()) {
-                // Numeric part: remove leading zeros
-                part.parse::<u32>().unwrap_or(0).to_string()
-            } else {
-                // Mixed part (contains both numbers and pre-release markers)
-                // Split on first non-digit character and normalize the numeric prefix
-                let chars = part.chars();
-                let mut numeric_prefix = String::new();
-                let mut rest = String::new();
-                let mut found_non_digit = false;
-
-                for ch in chars {
-                    if ch.is_ascii_digit() && !found_non_digit {
-                        numeric_prefix.push(ch);
-                    } else {
-                        found_non_digit = true;
-                        rest.push(ch);
-                    }
-                }
-
-                if numeric_prefix.is_empty() {
-                    part.to_string()
-                } else {
-                    let normalized_prefix = numeric_prefix.parse::<u32>().unwrap_or(0).to_string();
-                    format!("{normalized_prefix}{rest}")
-                }
-            }
-        })
-        .collect::<Vec<_>>()
-        .join(".")
-}
-
 pub fn setup_venv(build: &mut Build) -> Result<()> {
-    let extra_binary_exports = &["mypy", "ruff", "pytest", "protoc-gen-mypy"];
+    let extra_binary_exports = &[
+        "mypy",
+        "black",
+        "isort",
+        "pylint",
+        "pytest",
+        "protoc-gen-mypy",
+    ];
     build.add_action(
         "pyenv",
         PythonEnvironment {
@@ -123,14 +96,7 @@ impl BuildAction for BuildWheel {
     }
 
     fn files(&mut self, build: &mut impl FilesHandle) {
-        if std::env::var("OFFLINE_BUILD").ok().as_deref() == Some("1") {
-            let uv_path =
-                std::env::var("UV_BINARY").expect("UV_BINARY must be set in OFFLINE_BUILD mode");
-            build.add_inputs("uv", inputs![uv_path]);
-        } else {
         build.add_inputs("uv", inputs![":uv_binary"]);
-        }
-
         build.add_inputs("", &self.deps);
 
         // Set the project directory based on which package we're building
@@ -165,7 +131,14 @@ impl BuildAction for BuildWheel {
 
         let name = self.name;
 
-        let normalized_version = normalize_version(&self.version);
+        // Normalize version like hatchling does: remove leading zeros from version
+        // parts
+        let normalized_version = self
+            .version
+            .split('.')
+            .map(|part| part.parse::<u32>().unwrap_or(0).to_string())
+            .collect::<Vec<_>>()
+            .join(".");
 
         let wheel_path = format!("wheels/{name}-{normalized_version}-{tag}.whl");
         build.add_outputs("out", vec![wheel_path]);
@@ -195,26 +168,60 @@ pub fn check_python(build: &mut Build) -> Result<()> {
         },
     )?;
 
-    let ruff_folders = &["qt/aqt", "ftl", "pylib/tools", "tools", "python"];
+    add_pylint(build)?;
-    let ruff_deps = inputs![
+
-        glob!["{pylib,ftl,qt,python,tools}/**/*.py"],
+    Ok(())
-        ":pylib:anki",
+}
-        ":qt:aqt"
+
-    ];
+fn add_pylint(build: &mut Build) -> Result<()> {
+    // pylint does not support PEP420 implicit namespaces split across import paths,
+    // so we need to merge our pylib sources and generated files before invoking it,
+    // and add a top-level __init__.py
     build.add_action(
-        "check:ruff",
+        "check:pylint:copy_pylib",
-        RuffCheck {
+        RsyncFiles {
-            folders: ruff_folders,
+            inputs: inputs![":pylib:anki"],
-            deps: ruff_deps.clone(),
+            target_folder: "pylint/anki",
-            check_only: true,
+            strip_prefix: "$builddir/pylib/anki",
+            // avoid copying our large rsbridge binary
+            extra_args: "--links",
         },
     )?;
     build.add_action(
-        "fix:ruff",
+        "check:pylint:copy_pylib",
-        RuffCheck {
+        RsyncFiles {
-            folders: ruff_folders,
+            inputs: inputs![glob!["pylib/anki/**"]],
-            deps: ruff_deps,
+            target_folder: "pylint/anki",
-            check_only: false,
+            strip_prefix: "pylib/anki",
+            extra_args: "",
+        },
+    )?;
+    build.add_action(
+        "check:pylint:copy_pylib",
+        RunCommand {
+            command: ":pyenv:bin",
+            args: "$script $out",
+            inputs: hashmap! { "script" => inputs!["python/mkempty.py"] },
+            outputs: hashmap! { "out" => vec!["pylint/anki/__init__.py"] },
+        },
+    )?;
+    build.add_action(
+        "check:pylint",
+        PythonLint {
+            folders: &[
+                "$builddir/pylint/anki",
+                "qt/aqt",
+                "ftl",
+                "pylib/tools",
+                "tools",
+                "python",
+            ],
+            pylint_ini: inputs![".pylintrc"],
+            deps: inputs![
+                ":check:pylint:copy_pylib",
+                ":qt:aqt",
+                glob!("{pylib/tools,ftl,qt,python,tools}/**/*.py")
+            ],
         },
     )?;
 
@@ -227,19 +234,15 @@ struct Sphinx {
 
 impl BuildAction for Sphinx {
     fn command(&self) -> &str {
-        if std::env::var("OFFLINE_BUILD").ok().as_deref() == Some("1") {
+        if env::var("OFFLINE_BUILD").is_err() {
-            "$python python/sphinx/build.py"
-        } else {
             "$uv sync --extra sphinx && $python python/sphinx/build.py"
+        } else {
+            "$python python/sphinx/build.py"
         }
     }
 
     fn files(&mut self, build: &mut impl FilesHandle) {
-        if std::env::var("OFFLINE_BUILD").ok().as_deref() == Some("1") {
+        if env::var("OFFLINE_BUILD").is_err() {
-            let uv_path =
-                std::env::var("UV_BINARY").expect("UV_BINARY must be set in OFFLINE_BUILD mode");
-            build.add_inputs("uv", inputs![uv_path]);
-        } else {
             build.add_inputs("uv", inputs![":uv_binary"]);
             // Set environment variable to use the existing pyenv
             build.add_variable("pyenv_path", "$builddir/pyenv");
@@ -276,25 +279,3 @@ pub(crate) fn setup_sphinx(build: &mut Build) -> Result<()> {
     )?;
     Ok(())
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_normalize_version_basic() {
-        assert_eq!(normalize_version("1.2.3"), "1.2.3");
-        assert_eq!(normalize_version("01.02.03"), "1.2.3");
-        assert_eq!(normalize_version("1.0.0"), "1.0.0");
-    }
-
-    #[test]
-    fn test_normalize_version_with_prerelease() {
-        assert_eq!(normalize_version("1.2.3b1"), "1.2.3b1");
-        assert_eq!(normalize_version("01.02.03b1"), "1.2.3b1");
-        assert_eq!(normalize_version("1.0.0rc2"), "1.0.0rc2");
-        assert_eq!(normalize_version("2.1.0a3"), "2.1.0a3");
-        assert_eq!(normalize_version("1.2.3beta1"), "1.2.3beta1");
-        assert_eq!(normalize_version("1.2.3alpha1"), "1.2.3alpha1");
-    }
-}

build/configure/src/rust.rs

@@ -169,7 +169,7 @@ fn build_rsbridge(build: &mut Build) -> Result<()> {
 
 pub fn check_rust(build: &mut Build) -> Result<()> {
     let inputs = inputs![
-        glob!("{rslib/**,pylib/rsbridge/**,ftl/**,build/**,qt/launcher/**,tools/minilints/**}"),
+        glob!("{rslib/**,pylib/rsbridge/**,ftl/**,build/**,tools/workspace-hack/**}"),
         "Cargo.lock",
         "Cargo.toml",
         "rust-toolchain.toml",
@@ -247,7 +247,7 @@ pub fn check_minilints(build: &mut Build) -> Result<()> {
     let files = inputs![
         glob![
             "**/*.{py,rs,ts,svelte,mjs,md}",
-            "{node_modules,ts/.svelte-kit}/**"
+            "{node_modules,qt/bundle/PyOxidizer,ts/.svelte-kit}/**"
         ],
         "Cargo.lock"
     ];

build/ninja_gen/Cargo.toml

@@ -18,7 +18,6 @@ maplit.workspace = true
 num_cpus.workspace = true
 regex.workspace = true
 serde_json.workspace = true
-sha2.workspace = true
 walkdir.workspace = true
 which.workspace = true
 
@@ -31,11 +30,3 @@ reqwest = { workspace = true, features = ["blocking", "json", "rustls-tls"] }
 [[bin]]
 name = "update_uv"
 path = "src/bin/update_uv.rs"
-
-[[bin]]
-name = "update_protoc"
-path = "src/bin/update_protoc.rs"
-
-[[bin]]
-name = "update_node"
-path = "src/bin/update_node.rs"

build/ninja_gen/src/action.rs

@@ -49,46 +49,6 @@ pub trait BuildAction {
     }
 
     fn name(&self) -> &'static str {
-        std::any::type_name::<Self>()
+        std::any::type_name::<Self>().split("::").last().unwrap()
-            .split("::")
-            .last()
-            .unwrap()
-            .split('<')
-            .next()
-            .unwrap()
     }
 }
-
-#[cfg(test)]
-trait TestBuildAction {}
-
-#[cfg(test)]
-impl<T: TestBuildAction + ?Sized> BuildAction for T {
-    fn command(&self) -> &str {
-        "test"
-    }
-    fn files(&mut self, _build: &mut impl FilesHandle) {}
-}
-
-#[allow(dead_code, unused_variables)]
-#[test]
-fn should_strip_regions_in_type_name() {
-    struct Bare;
-    impl TestBuildAction for Bare {}
-    assert_eq!(Bare {}.name(), "Bare");
-
-    struct WithLifeTime<'a>(&'a str);
-    impl TestBuildAction for WithLifeTime<'_> {}
-    assert_eq!(WithLifeTime("test").name(), "WithLifeTime");
-
-    struct WithMultiLifeTime<'a, 'b>(&'a str, &'b str);
-    impl TestBuildAction for WithMultiLifeTime<'_, '_> {}
-    assert_eq!(
-        WithMultiLifeTime("test", "test").name(),
-        "WithMultiLifeTime"
-    );
-
-    struct WithGeneric<T>(T);
-    impl<T> TestBuildAction for WithGeneric<T> {}
-    assert_eq!(WithGeneric(3).name(), "WithGeneric");
-}

build/ninja_gen/src/archives.rs

@@ -67,7 +67,7 @@ impl Platform {
 }
 
 /// Append .exe to path if on Windows.
-pub fn with_exe(path: &str) -> Cow<'_, str> {
+pub fn with_exe(path: &str) -> Cow<str> {
     if cfg!(windows) {
         format!("{path}.exe").into()
     } else {

build/ninja_gen/src/bin/update_node.rs

@@ -1,268 +0,0 @@
-// Copyright: Ankitects Pty Ltd and contributors
-// License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
-
-use std::error::Error;
-use std::fs;
-use std::path::Path;
-
-use regex::Regex;
-use reqwest::blocking::Client;
-use serde_json::Value;
-
-#[derive(Debug)]
-struct NodeRelease {
-    version: String,
-    files: Vec<NodeFile>,
-}
-
-#[derive(Debug)]
-struct NodeFile {
-    filename: String,
-    url: String,
-}
-
-fn main() -> Result<(), Box<dyn Error>> {
-    let release_info = fetch_node_release_info()?;
-    let new_text = generate_node_archive_function(&release_info)?;
-    update_node_text(&new_text)?;
-    println!("Node.js archive function updated successfully!");
-    Ok(())
-}
-
-fn fetch_node_release_info() -> Result<NodeRelease, Box<dyn Error>> {
-    let client = Client::new();
-
-    // Get the Node.js release info
-    let response = client
-        .get("https://nodejs.org/dist/index.json")
-        .header("User-Agent", "anki-build-updater")
-        .send()?;
-
-    let releases: Vec<Value> = response.json()?;
-
-    // Find the latest LTS release
-    let latest = releases
-        .iter()
-        .find(|release| {
-            // LTS releases have a non-false "lts" field
-            release["lts"].as_str().is_some() && release["lts"] != false
-        })
-        .ok_or("No LTS releases found")?;
-
-    let version = latest["version"]
-        .as_str()
-        .ok_or("Version not found")?
-        .to_string();
-
-    let files = latest["files"]
-        .as_array()
-        .ok_or("Files array not found")?
-        .iter()
-        .map(|f| f.as_str().unwrap_or(""))
-        .collect::<Vec<_>>();
-
-    let lts_name = latest["lts"].as_str().unwrap_or("unknown");
-    println!("Found Node.js LTS version: {version} ({lts_name})");
-
-    // Map platforms to their expected file keys and full filenames
-    let platform_mapping = vec![
-        (
-            "linux-x64",
-            "linux-x64",
-            format!("node-{version}-linux-x64.tar.xz"),
-        ),
-        (
-            "linux-arm64",
-            "linux-arm64",
-            format!("node-{version}-linux-arm64.tar.xz"),
-        ),
-        (
-            "darwin-x64",
-            "osx-x64-tar",
-            format!("node-{version}-darwin-x64.tar.xz"),
-        ),
-        (
-            "darwin-arm64",
-            "osx-arm64-tar",
-            format!("node-{version}-darwin-arm64.tar.xz"),
-        ),
-        (
-            "win-x64",
-            "win-x64-zip",
-            format!("node-{version}-win-x64.zip"),
-        ),
-        (
-            "win-arm64",
-            "win-arm64-zip",
-            format!("node-{version}-win-arm64.zip"),
-        ),
-    ];
-
-    let mut node_files = Vec::new();
-
-    for (platform, file_key, filename) in platform_mapping {
-        // Check if this file exists in the release
-        if files.contains(&file_key) {
-            let url = format!("https://nodejs.org/dist/{version}/{filename}");
-            node_files.push(NodeFile {
-                filename: filename.clone(),
-                url,
-            });
-            println!("Found file for {platform}: {filename} (key: {file_key})");
-        } else {
-            return Err(
-                format!("File not found for {platform} (key: {file_key}): {filename}").into(),
-            );
-        }
-    }
-
-    Ok(NodeRelease {
-        version,
-        files: node_files,
-    })
-}
-
-fn generate_node_archive_function(release: &NodeRelease) -> Result<String, Box<dyn Error>> {
-    let client = Client::new();
-
-    // Fetch the SHASUMS256.txt file once
-    println!("Fetching SHA256 checksums...");
-    let shasums_url = format!("https://nodejs.org/dist/{}/SHASUMS256.txt", release.version);
-    let shasums_response = client
-        .get(&shasums_url)
-        .header("User-Agent", "anki-build-updater")
-        .send()?;
-    let shasums_text = shasums_response.text()?;
-
-    // Create a mapping from filename patterns to platform names - using the exact
-    // patterns we stored in files
-    let platform_mapping = vec![
-        ("linux-x64.tar.xz", "LinuxX64"),
-        ("linux-arm64.tar.xz", "LinuxArm"),
-        ("darwin-x64.tar.xz", "MacX64"),
-        ("darwin-arm64.tar.xz", "MacArm"),
-        ("win-x64.zip", "WindowsX64"),
-        ("win-arm64.zip", "WindowsArm"),
-    ];
-
-    let mut platform_blocks = Vec::new();
-
-    for (file_pattern, platform_name) in platform_mapping {
-        // Find the file that ends with this pattern
-        if let Some(file) = release
-            .files
-            .iter()
-            .find(|f| f.filename.ends_with(file_pattern))
-        {
-            // Find the SHA256 for this file
-            let sha256 = shasums_text
-                .lines()
-                .find(|line| line.contains(&file.filename))
-                .and_then(|line| line.split_whitespace().next())
-                .ok_or_else(|| format!("SHA256 not found for {}", file.filename))?;
-
-            println!(
-                "Found SHA256 for {}: {} => {}",
-                platform_name, file.filename, sha256
-            );
-
-            let block = format!(
-                "        Platform::{} => OnlineArchive {{\n            url: \"{}\",\n            sha256: \"{}\",\n        }},",
-                platform_name, file.url, sha256
-            );
-            platform_blocks.push(block);
-        } else {
-            return Err(format!(
-                "File not found for platform {platform_name}: no file ending with {file_pattern}"
-            )
-            .into());
-        }
-    }
-
-    let function = format!(
-        "pub fn node_archive(platform: Platform) -> OnlineArchive {{\n    match platform {{\n{}\n    }}\n}}",
-        platform_blocks.join("\n")
-    );
-
-    Ok(function)
-}
-
-fn update_node_text(new_function: &str) -> Result<(), Box<dyn Error>> {
-    let node_rs_content = read_node_rs()?;
-
-    // Regex to match the entire node_archive function with proper multiline
-    // matching
-    let re = Regex::new(
-        r"(?s)pub fn node_archive\(platform: Platform\) -> OnlineArchive \{.*?\n\s*\}\s*\n\s*\}",
-    )?;
-
-    let updated_content = re.replace(&node_rs_content, new_function);
-
-    write_node_rs(&updated_content)?;
-    Ok(())
-}
-
-fn read_node_rs() -> Result<String, Box<dyn Error>> {
-    // Use CARGO_MANIFEST_DIR to get the crate root, then find src/node.rs
-    let manifest_dir =
-        std::env::var("CARGO_MANIFEST_DIR").map_err(|_| "CARGO_MANIFEST_DIR not set")?;
-    let path = Path::new(&manifest_dir).join("src").join("node.rs");
-    Ok(fs::read_to_string(path)?)
-}
-
-fn write_node_rs(content: &str) -> Result<(), Box<dyn Error>> {
-    // Use CARGO_MANIFEST_DIR to get the crate root, then find src/node.rs
-    let manifest_dir =
-        std::env::var("CARGO_MANIFEST_DIR").map_err(|_| "CARGO_MANIFEST_DIR not set")?;
-    let path = Path::new(&manifest_dir).join("src").join("node.rs");
-    fs::write(path, content)?;
-    Ok(())
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_regex_replacement() {
-        let sample_content = r#"Some other code
-pub fn node_archive(platform: Platform) -> OnlineArchive {
-    match platform {
-        Platform::LinuxX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-linux-x64.tar.xz",
-            sha256: "old_hash",
-        },
-        Platform::MacX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-darwin-x64.tar.xz",
-            sha256: "old_hash",
-        },
-    }
-}
-
-More code here"#;
-
-        let new_function = r#"pub fn node_archive(platform: Platform) -> OnlineArchive {
-    match platform {
-        Platform::LinuxX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v21.0.0/node-v21.0.0-linux-x64.tar.xz",
-            sha256: "new_hash",
-        },
-        Platform::MacX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v21.0.0/node-v21.0.0-darwin-x64.tar.xz",
-            sha256: "new_hash",
-        },
-    }
-}"#;
-
-        let re = Regex::new(
-            r"(?s)pub fn node_archive\(platform: Platform\) -> OnlineArchive \{.*?\n\s*\}\s*\n\s*\}"
-        ).unwrap();
-
-        let result = re.replace(sample_content, new_function);
-        assert!(result.contains("v21.0.0"));
-        assert!(result.contains("new_hash"));
-        assert!(!result.contains("old_hash"));
-        assert!(result.contains("Some other code"));
-        assert!(result.contains("More code here"));
-    }
-}

build/ninja_gen/src/bin/update_protoc.rs

@@ -1,125 +0,0 @@
-// Copyright: Ankitects Pty Ltd and contributors
-// License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
-
-use std::error::Error;
-use std::fs;
-use std::path::Path;
-
-use regex::Regex;
-use reqwest::blocking::Client;
-use serde_json::Value;
-use sha2::Digest;
-use sha2::Sha256;
-
-fn fetch_protoc_release_info() -> Result<String, Box<dyn Error>> {
-    let client = Client::new();
-
-    println!("Fetching latest protoc release info from GitHub...");
-    // Fetch latest release info
-    let response = client
-        .get("https://api.github.com/repos/protocolbuffers/protobuf/releases/latest")
-        .header("User-Agent", "Anki-Build-Script")
-        .send()?;
-
-    let release_info: Value = response.json()?;
-    let assets = release_info["assets"]
-        .as_array()
-        .expect("assets should be an array");
-
-    // Map platform names to their corresponding asset patterns
-    let platform_patterns = [
-        ("LinuxX64", "linux-x86_64"),
-        ("LinuxArm", "linux-aarch_64"),
-        ("MacX64", "osx-universal_binary"), // Mac uses universal binary for both
-        ("MacArm", "osx-universal_binary"),
-        ("WindowsX64", "win64"), // Windows uses x86 binary for both archs
-        ("WindowsArm", "win64"),
-    ];
-
-    let mut match_blocks = Vec::new();
-
-    for (platform, pattern) in platform_patterns {
-        // Find the asset matching the platform pattern
-        let asset = assets.iter().find(|asset| {
-            let name = asset["name"].as_str().unwrap_or("");
-            name.starts_with("protoc-") && name.contains(pattern) && name.ends_with(".zip")
-        });
-
-        if asset.is_none() {
-            eprintln!("No asset found for platform {platform} pattern {pattern}");
-            continue;
-        }
-
-        let asset = asset.unwrap();
-        let download_url = asset["browser_download_url"].as_str().unwrap();
-        let asset_name = asset["name"].as_str().unwrap();
-
-        // Download the file and calculate SHA256 locally
-        println!("Downloading and checksumming {asset_name} for {platform}...");
-        let response = client
-            .get(download_url)
-            .header("User-Agent", "Anki-Build-Script")
-            .send()?;
-
-        let bytes = response.bytes()?;
-        let mut hasher = Sha256::new();
-        hasher.update(&bytes);
-        let sha256 = format!("{:x}", hasher.finalize());
-
-        // Handle platform-specific match patterns
-        let match_pattern = match platform {
-            "MacX64" => "Platform::MacX64 | Platform::MacArm",
-            "MacArm" => continue, // Skip MacArm since it's handled with MacX64
-            "WindowsX64" => "Platform::WindowsX64 | Platform::WindowsArm",
-            "WindowsArm" => continue, // Skip WindowsArm since it's handled with WindowsX64
-            _ => &format!("Platform::{platform}"),
-        };
-
-        match_blocks.push(format!(
-            "        {match_pattern} => {{\n            OnlineArchive {{\n                url: \"{download_url}\",\n                sha256: \"{sha256}\",\n            }}\n        }}"
-        ));
-    }
-
-    Ok(format!(
-        "pub fn protoc_archive(platform: Platform) -> OnlineArchive {{\n    match platform {{\n{}\n    }}\n}}",
-        match_blocks.join(",\n")
-    ))
-}
-
-fn read_protobuf_rs() -> Result<String, Box<dyn Error>> {
-    let manifest_dir = std::env::var("CARGO_MANIFEST_DIR").unwrap_or_else(|_| ".".to_string());
-    let path = Path::new(&manifest_dir).join("src/protobuf.rs");
-    println!("Reading {}", path.display());
-    let content = fs::read_to_string(path)?;
-    Ok(content)
-}
-
-fn update_protoc_text(old_text: &str, new_protoc_text: &str) -> Result<String, Box<dyn Error>> {
-    let re =
-        Regex::new(r"(?ms)^pub fn protoc_archive\(platform: Platform\) -> OnlineArchive \{.*?\n\}")
-            .unwrap();
-    if !re.is_match(old_text) {
-        return Err("Could not find protoc_archive function block to replace".into());
-    }
-    let new_content = re.replace(old_text, new_protoc_text).to_string();
-    println!("Original lines: {}", old_text.lines().count());
-    println!("Updated lines: {}", new_content.lines().count());
-    Ok(new_content)
-}
-
-fn write_protobuf_rs(content: &str) -> Result<(), Box<dyn Error>> {
-    let manifest_dir = std::env::var("CARGO_MANIFEST_DIR").unwrap_or_else(|_| ".".to_string());
-    let path = Path::new(&manifest_dir).join("src/protobuf.rs");
-    println!("Writing to {}", path.display());
-    fs::write(path, content)?;
-    Ok(())
-}
-
-fn main() -> Result<(), Box<dyn Error>> {
-    let new_protoc_archive = fetch_protoc_release_info()?;
-    let content = read_protobuf_rs()?;
-    let updated_content = update_protoc_text(&content, &new_protoc_archive)?;
-    write_protobuf_rs(&updated_content)?;
-    println!("Successfully updated protoc_archive function in protobuf.rs");
-    Ok(())
-}

build/ninja_gen/src/bin/update_uv.rs

@@ -53,7 +53,7 @@ fn fetch_uv_release_info() -> Result<String, Box<dyn Error>> {
         // Find the corresponding .sha256 or .sha256sum asset
         let sha_asset = assets.iter().find(|a| {
             let name = a["name"].as_str().unwrap_or("");
-            name == format!("{asset_name}.sha256") || name == format!("{asset_name}.sha256sum")
+            name == format!("{}.sha256", asset_name) || name == format!("{}.sha256sum", asset_name)
         });
         if sha_asset.is_none() {
             eprintln!("No sha256 asset found for {asset_name}");
@@ -71,7 +71,8 @@ fn fetch_uv_release_info() -> Result<String, Box<dyn Error>> {
         let sha256 = sha_text.split_whitespace().next().unwrap_or("");
 
         match_blocks.push(format!(
-            "        Platform::{platform} => {{\n            OnlineArchive {{\n                url: \"{download_url}\",\n                sha256: \"{sha256}\",\n            }}\n        }}"
+            "        Platform::{} => {{\n            OnlineArchive {{\n                url: \"{}\",\n                sha256: \"{}\",\n            }}\n        }}",
+            platform, download_url, sha256
         ));
     }
 
@@ -134,7 +135,10 @@ mod tests {
         assert_eq!(
             updated_lines,
             original_lines - EXPECTED_LINES_REMOVED,
-            "Expected line count to decrease by exactly {EXPECTED_LINES_REMOVED} lines (original: {original_lines}, updated: {updated_lines})"
+            "Expected line count to decrease by exactly {} lines (original: {}, updated: {})",
+            EXPECTED_LINES_REMOVED,
+            original_lines,
+            updated_lines
         );
     }
 }

build/ninja_gen/src/build.rs

@@ -300,7 +300,7 @@ impl BuildStatement<'_> {
 
         writeln!(buf, "build {outputs_str}: {action_name} {inputs_str}").unwrap();
         for (key, value) in self.variables.iter().sorted() {
-            writeln!(buf, "  {key} = {value}").unwrap();
+            writeln!(buf, "  {key} = {}", value).unwrap();
         }
         writeln!(buf).unwrap();
 
@@ -476,7 +476,7 @@ impl FilesHandle for BuildStatement<'_> {
         let outputs = outputs.into_iter().map(|v| {
             let v = v.as_ref();
             let v = if !v.starts_with("$builddir/") && !v.starts_with("$builddir\\") {
-                format!("$builddir/{v}")
+                format!("$builddir/{}", v)
             } else {
                 v.to_owned()
             };

build/ninja_gen/src/cargo.rs

@@ -162,7 +162,7 @@ impl BuildAction for CargoTest {
             "cargo-nextest",
             CargoInstall {
                 binary_name: "cargo-nextest",
-                args: "cargo-nextest --version 0.9.99 --locked --no-default-features --features default-no-update",
+                args: "cargo-nextest --version 0.9.57 --locked",
             },
         )?;
         setup_flags(build)

build/ninja_gen/src/node.rs

@@ -19,28 +19,28 @@ use crate::input::BuildInput;
 pub fn node_archive(platform: Platform) -> OnlineArchive {
     match platform {
         Platform::LinuxX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v22.17.0/node-v22.17.0-linux-x64.tar.xz",
+            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-linux-x64.tar.xz",
-            sha256: "325c0f1261e0c61bcae369a1274028e9cfb7ab7949c05512c5b1e630f7e80e12",
+            sha256: "822780369d0ea309e7d218e41debbd1a03f8cdf354ebf8a4420e89f39cc2e612",
         },
         Platform::LinuxArm => OnlineArchive {
-            url: "https://nodejs.org/dist/v22.17.0/node-v22.17.0-linux-arm64.tar.xz",
+            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-linux-arm64.tar.xz",
-            sha256: "140aee84be6774f5fb3f404be72adbe8420b523f824de82daeb5ab218dab7b18",
+            sha256: "f6df68c6793244071f69023a9b43a0cf0b13d65cbe86d55925c28e4134d9aafb",
         },
         Platform::MacX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v22.17.0/node-v22.17.0-darwin-x64.tar.xz",
+            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-darwin-x64.tar.xz",
-            sha256: "f79de1f64df4ac68493a344bb5ab7d289d0275271e87b543d1278392c9de778a",
+            sha256: "d4b4ab81ebf1f7aab09714f834992f27270ad0079600da00c8110f8950ca6c5a",
         },
         Platform::MacArm => OnlineArchive {
-            url: "https://nodejs.org/dist/v22.17.0/node-v22.17.0-darwin-arm64.tar.xz",
+            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-darwin-arm64.tar.xz",
-            sha256: "cc9cc294eaf782dd93c8c51f460da610cc35753c6a9947411731524d16e97914",
+            sha256: "f18a7438723d48417f5e9be211a2f3c0520ffbf8e02703469e5153137ca0f328",
         },
         Platform::WindowsX64 => OnlineArchive {
-            url: "https://nodejs.org/dist/v22.17.0/node-v22.17.0-win-x64.zip",
+            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-win-x64.zip",
-            sha256: "721ab118a3aac8584348b132767eadf51379e0616f0db802cc1e66d7f0d98f85",
+            sha256: "893115cd92ad27bf178802f15247115e93c0ef0c753b93dca96439240d64feb5",
         },
         Platform::WindowsArm => OnlineArchive {
-            url: "https://nodejs.org/dist/v22.17.0/node-v22.17.0-win-arm64.zip",
+            url: "https://nodejs.org/dist/v20.11.0/node-v20.11.0-win-arm64.zip",
-            sha256: "78355dc9ca117bb71d3f081e4b1b281855e2b134f3939bb0ca314f7567b0e621",
+            sha256: "89c1f7034dcd6ff5c17f2af61232a96162a1902f862078347dcf274a938b6142",
         },
     }
 }
@@ -98,7 +98,7 @@ impl BuildAction for YarnInstall<'_> {
     }
 }
 
-fn with_cmd_ext(bin: &str) -> Cow<'_, str> {
+fn with_cmd_ext(bin: &str) -> Cow<str> {
     if cfg!(windows) {
         format!("{bin}.cmd").into()
     } else {

build/ninja_gen/src/protobuf.rs

@@ -21,26 +21,26 @@ pub fn protoc_archive(platform: Platform) -> OnlineArchive {
     match platform {
         Platform::LinuxX64 => {
             OnlineArchive {
-                url: "https://github.com/protocolbuffers/protobuf/releases/download/v31.1/protoc-31.1-linux-x86_64.zip",
+                url: "https://github.com/protocolbuffers/protobuf/releases/download/v21.8/protoc-21.8-linux-x86_64.zip",
-                sha256: "96553041f1a91ea0efee963cb16f462f5985b4d65365f3907414c360044d8065",
+                sha256: "f90d0dd59065fef94374745627336d622702b67f0319f96cee894d41a974d47a",
+            }
         }
-        },
         Platform::LinuxArm => {
             OnlineArchive {
-                url: "https://github.com/protocolbuffers/protobuf/releases/download/v31.1/protoc-31.1-linux-aarch_64.zip",
+                url: "https://github.com/protocolbuffers/protobuf/releases/download/v21.8/protoc-21.8-linux-aarch_64.zip",
-                sha256: "6c554de11cea04c56ebf8e45b54434019b1cd85223d4bbd25c282425e306ecc2",
+                sha256: "f3d8eb5839d6186392d8c7b54fbeabbb6fcdd90618a500b77cb2e24faa245cad",
+            }
         }
-        },
         Platform::MacX64 | Platform::MacArm => {
             OnlineArchive {
-                url: "https://github.com/protocolbuffers/protobuf/releases/download/v31.1/protoc-31.1-osx-universal_binary.zip",
+                url: "https://github.com/protocolbuffers/protobuf/releases/download/v21.8/protoc-21.8-osx-universal_binary.zip",
-                sha256: "99ea004549c139f46da5638187a85bbe422d78939be0fa01af1aa8ab672e395f",
+                sha256: "e3324d3bc2e9bc967a0bec2472e0ec73b26f952c7c87f2403197414f780c3c6c",
+            }
         }
-        },
         Platform::WindowsX64 | Platform::WindowsArm => {
             OnlineArchive {
-                url: "https://github.com/protocolbuffers/protobuf/releases/download/v31.1/protoc-31.1-win64.zip",
+                url: "https://github.com/protocolbuffers/protobuf/releases/download/v21.8/protoc-21.8-win64.zip",
-                sha256: "70381b116ab0d71cb6a5177d9b17c7c13415866603a0fd40d513dafe32d56c35",
+                sha256: "3657053024faa439ff5f8c1dd2ee06bac0f9b9a3d660e99944f015a7451e87ec",
             }
         }
     }

build/ninja_gen/src/python.rs

@@ -148,7 +148,7 @@ impl BuildAction for PythonEnvironment {
             // Add --python flag to extra_args if PYTHON_BINARY is set
             let mut args = self.extra_args.to_string();
             if let Ok(python_binary) = env::var("PYTHON_BINARY") {
-                args = format!("--python {python_binary} {args}");
+                args = format!("--python {} {}", python_binary, args);
             }
             build.add_variable("extra_args", args);
         }
@@ -159,10 +159,6 @@ impl BuildAction for PythonEnvironment {
         }
         build.add_output_stamp(format!("{}/.stamp", self.venv_folder));
     }
-
-    fn check_output_timestamps(&self) -> bool {
-        true
-    }
 }
 
 pub struct PythonTypecheck {
@@ -193,19 +189,31 @@ impl BuildAction for PythonTypecheck {
 struct PythonFormat<'a> {
     pub inputs: &'a BuildInput,
     pub check_only: bool,
+    pub isort_ini: &'a BuildInput,
 }
 
 impl BuildAction for PythonFormat<'_> {
     fn command(&self) -> &str {
-        "$ruff format $mode $in && $ruff check --select I --fix $in"
+        "$black -t py39 -q $check --color $in && $
+         $isort --color --settings-path $isort_ini $check $in"
     }
 
     fn files(&mut self, build: &mut impl crate::build::FilesHandle) {
         build.add_inputs("in", self.inputs);
-        build.add_inputs("ruff", inputs![":pyenv:ruff"]);
+        build.add_inputs("black", inputs![":pyenv:black"]);
+        build.add_inputs("isort", inputs![":pyenv:isort"]);
 
         let hash = simple_hash(self.inputs);
-        build.add_variable("mode", if self.check_only { "--check" } else { "" });
+        build.add_env_var("BLACK_CACHE_DIR", "out/python/black.cache.{hash}");
+        build.add_inputs("isort_ini", self.isort_ini);
+        build.add_variable(
+            "check",
+            if self.check_only {
+                "--diff --check"
+            } else {
+                ""
+            },
+        );
 
         build.add_output_stamp(format!(
             "tests/python_format.{}.{hash}",
@@ -215,11 +223,13 @@ impl BuildAction for PythonFormat<'_> {
 }
 
 pub fn python_format(build: &mut Build, group: &str, inputs: BuildInput) -> Result<()> {
+    let isort_ini = &inputs![".isort.cfg"];
     build.add_action(
         format!("check:format:python:{group}"),
         PythonFormat {
             inputs: &inputs,
             check_only: true,
+            isort_ini,
         },
     )?;
 
@@ -228,39 +238,34 @@ pub fn python_format(build: &mut Build, group: &str, inputs: BuildInput) -> Resu
         PythonFormat {
             inputs: &inputs,
             check_only: false,
+            isort_ini,
         },
     )?;
     Ok(())
 }
 
-pub struct RuffCheck {
+pub struct PythonLint {
     pub folders: &'static [&'static str],
+    pub pylint_ini: BuildInput,
     pub deps: BuildInput,
-    pub check_only: bool,
 }
 
-impl BuildAction for RuffCheck {
+impl BuildAction for PythonLint {
     fn command(&self) -> &str {
-        "$ruff check $folders $mode"
+        "$pylint --rcfile $pylint_ini -sn -j $cpus $folders"
     }
 
     fn files(&mut self, build: &mut impl crate::build::FilesHandle) {
         build.add_inputs("", &self.deps);
-        build.add_inputs("", inputs![".ruff.toml"]);
+        build.add_inputs("pylint", inputs![":pyenv:pylint"]);
-        build.add_inputs("ruff", inputs![":pyenv:ruff"]);
+        build.add_inputs("pylint_ini", &self.pylint_ini);
         build.add_variable("folders", self.folders.join(" "));
-        build.add_variable(
+        // On a 16 core system, values above 10 do not improve wall clock time,
-            "mode",
+        // but waste extra cores that could be working on other tests.
-            if self.check_only {
+        build.add_variable("cpus", num_cpus::get().min(10).to_string());
-                ""
-            } else {
-                "--fix --unsafe-fixes"
-            },
-        );
 
         let hash = simple_hash(&self.deps);
-        let kind = if self.check_only { "check" } else { "fix" };
+        build.add_output_stamp(format!("tests/python_lint.{hash}"));
-        build.add_output_stamp(format!("tests/python_ruff.{kind}.{hash}"));
     }
 }
 

build/ninja_gen/src/render.rs

@@ -30,12 +30,12 @@ impl Build {
         )
         .unwrap();
         for (key, value) in &self.variables {
-            writeln!(&mut buf, "{key} = {value}").unwrap();
+            writeln!(&mut buf, "{} = {}", key, value).unwrap();
         }
         buf.push('\n');
 
         for (key, value) in &self.pools {
-            writeln!(&mut buf, "pool {key}\n  depth = {value}").unwrap();
+            writeln!(&mut buf, "pool {}\n  depth = {}", key, value).unwrap();
         }
         buf.push('\n');
 

build/runner/src/archive.rs

@@ -65,7 +65,7 @@ fn sha2_data(data: &[u8]) -> String {
     let mut digest = sha2::Sha256::new();
     digest.update(data);
     let result = digest.finalize();
-    format!("{result:x}")
+    format!("{:x}", result)
 }
 
 enum CompressionKind {

build/runner/src/build.rs

@@ -67,10 +67,7 @@ pub fn run_build(args: BuildArgs) {
             "MYPY_CACHE_DIR",
             build_root.join("tests").join("mypy").into_string(),
         )
-        .env(
+        .env("PYTHONPYCACHEPREFIX", build_root.join("pycache"))
-            "PYTHONPYCACHEPREFIX",
-            std::path::absolute(build_root.join("pycache")).unwrap(),
-        )
         // commands will not show colors by default, as we do not provide a tty
         .env("FORCE_COLOR", "1")
         .env("MYPY_FORCE_COLOR", "1")
@@ -138,7 +135,7 @@ fn setup_build_root() -> Utf8PathBuf {
             true
         };
         if create {
-            println!("Switching build root to {new_target}");
+            println!("Switching build root to {}", new_target);
             std::os::unix::fs::symlink(new_target, build_root).unwrap();
         }
     }

build/runner/src/pyenv.rs

@@ -32,19 +32,10 @@ pub fn setup_pyenv(args: PyenvArgs) {
         }
     }
 
-    let mut command = Command::new(args.uv_bin);
-
-    // remove UV_* environment variables to avoid interference
-    for (key, _) in std::env::vars() {
-        if key.starts_with("UV_") || key == "VIRTUAL_ENV" {
-            command.env_remove(key);
-        }
-    }
-
     run_command(
-        command
+        Command::new(args.uv_bin)
             .env("UV_PROJECT_ENVIRONMENT", args.pyenv_folder.clone())
-            .args(["sync", "--locked", "--no-config"])
+            .args(["sync", "--frozen"])
             .args(args.extra_args),
     );
 

build/runner/src/run.rs

@@ -1,6 +1,7 @@
 // Copyright: Ankitects Pty Ltd and contributors
 // License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+use std::io::ErrorKind;
 use std::process::Command;
 
 use anki_io::create_dir_all;
@@ -43,7 +44,7 @@ fn split_env(s: &str) -> Result<(String, String), std::io::Error> {
     if let Some((k, v)) = s.split_once('=') {
         Ok((k.into(), v.into()))
     } else {
-        Err(std::io::Error::other("invalid env var"))
+        Err(std::io::Error::new(ErrorKind::Other, "invalid env var"))
     }
 }
 
@@ -83,7 +84,7 @@ fn split_args(args: Vec<String>) -> Vec<Vec<String>> {
 
 pub fn run_command(command: &mut Command) {
     if let Err(err) = command.ensure_success() {
-        println!("{err}");
+        println!("{}", err);
         std::process::exit(1);
     }
 }

build/runner/src/yarn.rs

@@ -28,11 +28,7 @@ pub fn setup_yarn(args: YarnArgs) {
                 .arg("--ignore-scripts"),
         );
     } else {
-        run_command(
+        run_command(Command::new(&args.yarn_bin).arg("install"));
-            Command::new(&args.yarn_bin)
-                .arg("install")
-                .arg("--immutable"),
-        );
     }
 
     std::fs::write(args.stamp, b"").unwrap();

docs/development.md

@@ -85,7 +85,7 @@ When formatting issues are reported, they can be fixed with
 ./ninja format
 ```
 
-## Fixing ruff/eslint/copyright header issues
+## Fixing eslint/copyright header issues
 
 ```
 ./ninja fix
@@ -190,10 +190,13 @@ in the collection2.log file will also be printed on stdout.
 
 If ANKI_PROFILE_CODE is set, Python profiling data will be written on exit.
 
-# Installer/launcher
+# Binary Bundles
 
-- The anki-release package is created/published with the scripts in qt/release.
+Anki's official binary packages are created with `./ninja bundle`. The bundling
-- The installer/launcher is created with the build scripts in qt/launcher/{platform}.
+process was created specifically for the official builds, and is provided as-is;
+we are unfortunately not able to provide assistance with any issues you may run
+into when using it. You'll need to run
+`git submodule update --checkout qt/bundle/PyOxidizer` first.
 
 ## Mixing development and study
 

docs/docker/Dockerfile

@@ -1,78 +1,35 @@
-# This is a user-contributed Dockerfile. No official support is available.
+# This Dockerfile uses three stages.
+#   1. Compile anki (and dependencies) and build python wheels.
+#   2. Create a virtual environment containing anki and its dependencies.
+#   3. Create a final image that only includes anki's virtual environment and required
+#      system packages.
 
+ARG PYTHON_VERSION="3.9"
 ARG DEBIAN_FRONTEND="noninteractive"
 
-FROM ubuntu:24.04 AS build
+# Build anki.
+FROM python:$PYTHON_VERSION AS build
+RUN curl -fsSL https://github.com/bazelbuild/bazelisk/releases/download/v1.7.4/bazelisk-linux-amd64 \
+    > /usr/local/bin/bazel \
+    && chmod +x /usr/local/bin/bazel \
+    # Bazel expects /usr/bin/python
+    && ln -s /usr/local/bin/python /usr/bin/python
 WORKDIR /opt/anki
-ENV PYTHON_VERSION="3.13"
-
-
-# System deps
-RUN apt-get update && apt-get install -y --no-install-recommends \
-    curl \
-    git \
-    build-essential \
-    pkg-config \
-    libssl-dev \
-    libbz2-dev \
-    libreadline-dev \
-    libsqlite3-dev \
-    libffi-dev \
-    zlib1g-dev \
-    liblzma-dev \
-    ca-certificates \
-    ninja-build \
-    rsync \
-    libglib2.0-0 \
-    libgl1 \
-    libx11-6 \
-    libxext6 \
-    libxrender1 \
-    libxkbcommon0 \
-    libxkbcommon-x11-0 \
-    libxcb1 \
-    libxcb-render0 \
-    libxcb-shm0 \
-    libxcb-icccm4 \
-    libxcb-image0 \
-    libxcb-keysyms1 \
-    libxcb-randr0 \
-    libxcb-shape0 \
-    libxcb-xfixes0 \
-    libxcb-xinerama0 \
-    libxcb-xinput0 \
-    libsm6 \
-    libice6 \
-    && rm -rf /var/lib/apt/lists/*
-
-# install rust with rustup
-RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
-ENV PATH="/root/.cargo/bin:${PATH}"
-
-# Install uv and Python 3.13 with uv
-RUN curl -LsSf https://astral.sh/uv/install.sh | sh \
-    && ln -s /root/.local/bin/uv /usr/local/bin/uv
-ENV PATH="/root/.local/bin:${PATH}"
-
-RUN uv python install ${PYTHON_VERSION} --default
-
 COPY . .
-
+# Build python wheels.
 RUN ./tools/build
 
-
 # Install pre-compiled Anki.
-FROM python:3.13-slim AS installer
+FROM python:${PYTHON_VERSION}-slim as installer
 WORKDIR /opt/anki/
-COPY --from=build /opt/anki/out/wheels/ wheels/
+COPY --from=build /opt/anki/wheels/ wheels/
 # Use virtual environment.
 RUN python -m venv venv \
     && ./venv/bin/python -m pip install --no-cache-dir setuptools wheel \
     && ./venv/bin/python -m pip install --no-cache-dir /opt/anki/wheels/*.whl
 
-
 # We use another build stage here so we don't include the wheels in the final image.
-FROM python:3.13-slim AS final
+FROM python:${PYTHON_VERSION}-slim as final
 COPY --from=installer /opt/anki/venv /opt/anki/venv
 ENV PATH=/opt/anki/venv/bin:$PATH
 # Install run-time dependencies.
@@ -102,9 +59,9 @@ RUN apt-get update \
     libxrender1 \
     libxtst6 \
     && rm -rf /var/lib/apt/lists/*
-
 # Add non-root user.
 RUN useradd --create-home anki
 USER anki
 WORKDIR /work
 ENTRYPOINT ["/opt/anki/venv/bin/anki"]
+LABEL maintainer="Jakub Kaczmarzyk <jakub.kaczmarzyk@gmail.com>"

docs/linux.md

@@ -51,8 +51,13 @@ Anki requires a recent glibc.
 
 If you are using a distro that uses musl, Anki will not work.
 
-You can use your system's Qt libraries if they are Qt 6.2 or later, if
+If your glibc version is 2.35+ on AMD64 or 2.39+ on ARM64, you can skip the rest of this section.
-you wish. After installing the system libraries (eg:
+
+If your system has an older glibc, you won't be able to use the PyQt wheels that are
+available in pip/PyPy, and will need to use your system-installed PyQt instead.
+Your distro will also need to have Python 3.9 or later.
+
+After installing the system libraries (eg:
 'sudo apt install python3-pyqt6.qt{quick,webengine} python3-venv pyqt6-dev-tools'),
 find the place they are installed (eg '/usr/lib/python3/dist-packages'). On modern Ubuntu, you'll
 also need 'sudo apt remove python3-protobuf'. Then before running any commands like './run', tell Anki where
@@ -63,6 +68,12 @@ export PYTHONPATH=/usr/lib/python3/dist-packages
 export PYTHON_BINARY=/usr/bin/python3
 ```
 
+There are a few things to be aware of:
+
+- You should use ./run and not tools/run-qt5\*, even if your system libraries are Qt5.
+- If your system libraries are Qt5, when creating an aqt wheel, the wheel will not work
+  on Qt6 environments.
+
 ## Packaging considerations
 
 Python, node and protoc are downloaded as part of the build. You can optionally define

docs/protobuf.md

@@ -98,6 +98,12 @@ should preferably be assigned a number between 1 and 15. If a message contains
 
 Protobuf has an official Python implementation with an extensive [reference](https://developers.google.com/protocol-buffers/docs/reference/python-generated).
 
+- Every message used in aqt or pylib must be added to the respective `.pylintrc`
+  to avoid failing type checks. The unqualified protobuf message's name must be
+  used, not an alias from `collection.py` for example. This should be taken into
+  account when choosing a message name in order to prevent skipping typechecking
+  a Python class of the same name.
+
 ### Typescript
 
 Anki uses [protobuf-es](https://github.com/bufbuild/protobuf-es), which offers

ftl/core-repo

@@ -1 +1 @@
-Subproject commit 480ef0da728c7ea3485c58529ae7ee02be3e5dba
+Subproject commit 78412ce163d4dc50dd82f5b27cde3119086a2eb7

ftl/core/card-templates.ftl

@@ -60,6 +60,7 @@ card-templates-this-will-create-card-proceed =
     }
 card-templates-type-boxes-warning = Only one typing box per card template is supported.
 card-templates-restore-to-default = Restore to Default
-card-templates-restore-to-default-confirmation = This will reset all fields and templates in this note type to their default values, removing any extra fields/templates and their content, and any custom styling. Do you wish to proceed?
+card-templates-restore-to-default-confirmation = This will reset all fields and templates in this note type to their default
+    values, removing any extra fields/templates and their content, and any custom styling. Do you wish to proceed?
 card-templates-restored-to-default = Note type has been restored to its original state.
 

ftl/core/database-check.ftl

@@ -5,11 +5,6 @@ database-check-card-properties =
         [one] Fixed { $count } invalid card property.
        *[other] Fixed { $count } invalid card properties.
     }
-database-check-card-last-review-time-empty =
-    { $count ->
-        [one] Added last review time to { $count } card.
-       *[other] Added last review time to { $count } cards.
-    }
 database-check-missing-templates =
     { $count ->
         [one] Deleted { $count } card with missing template.

ftl/core/deck-config.ftl

@@ -307,8 +307,7 @@ deck-config-new-interval-tooltip = The multiplier applied to a review interval w
 deck-config-minimum-interval-tooltip = The minimum interval given to a review card after answering `Again`.
 deck-config-custom-scheduling = Custom scheduling
 deck-config-custom-scheduling-tooltip = Affects the entire collection. Use at your own risk!
-
+# Easy Days section
-## Easy Days section.
 
 deck-config-easy-days-title = Easy Days
 deck-config-easy-days-monday = Mon
@@ -384,6 +383,8 @@ deck-config-which-deck = Which deck would you like to display options for?
 deck-config-updating-cards = Updating cards: { $current_cards_count }/{ $total_cards_count }...
 deck-config-invalid-parameters = The provided FSRS parameters are invalid. Leave them blank to use the default parameters.
 deck-config-not-enough-history = Insufficient review history to perform this operation.
+deck-config-unable-to-determine-desired-retention =
+    Unable to determine a minimum recommended retention.
 deck-config-must-have-400-reviews =
     { $count ->
         [one] Only { $count } review was found.
@@ -392,9 +393,10 @@ deck-config-must-have-400-reviews =
 # Numbers that control how aggressively the FSRS algorithm schedules cards
 deck-config-weights = FSRS parameters
 deck-config-compute-optimal-weights = Optimize FSRS parameters
+deck-config-compute-minimum-recommended-retention = Minimum recommended retention
 deck-config-optimize-button = Optimize Current Preset
-# Indicates that a given function or label, provided via the "text" variable, operates slowly.
 deck-config-slow-suffix = { $text } (slow)
+deck-config-health-check = Check health when optimizing
 deck-config-compute-button = Compute
 deck-config-ignore-before = Ignore cards reviewed before
 deck-config-time-to-optimize = It's been a while - using the Optimize All Presets button is recommended.
@@ -404,6 +406,7 @@ deck-config-historical-retention = Historical retention
 deck-config-smaller-is-better = Smaller numbers indicate a better fit to your review history.
 deck-config-steps-too-large-for-fsrs = When FSRS is enabled, steps of 1 day or more are not recommended.
 deck-config-get-params = Get Params
+deck-config-predicted-minimum-recommended-retention = Minimum recommended retention: { $num }
 deck-config-complete = { $num }% complete.
 deck-config-iterations = Iteration: { $count }...
 deck-config-reschedule-cards-on-change = Reschedule cards on change
@@ -421,8 +424,6 @@ deck-config-desired-retention-tooltip =
     less frequently, and you will forget more of them. Be conservative when adjusting this - higher
     values will greatly increase your workload, and lower values can be demoralizing when you forget
     a lot of material.
-deck-config-desired-retention-tooltip2 = 
-    The workload values provided by the info box are a rough approximation. For a greater level of accuracy, use the simulator.
 deck-config-historical-retention-tooltip =
     When some of your review history is missing, FSRS needs to fill in the gaps. By default, it will
     assume that when you did those old reviews, you remembered 90% of the material. If your old retention
@@ -464,7 +465,12 @@ deck-config-compute-optimal-weights-tooltip2 =
     By default, parameters will be calculated from the review history of all decks using the current preset. You can
     optionally adjust the search before calculating the parameters, if you'd like to alter which cards are used for
     optimizing the parameters.
-
+deck-config-compute-optimal-retention-tooltip4 =
+    This tool will attempt to find the desired retention value 
+    that will lead to the most material learnt, in the least amount of time. The calculated number can serve as a reference
+    when deciding what to set your desired retention to. You may wish to choose a higher desired retention if you’re 
+    willing to invest more study time to achieve it. Setting your desired retention lower than the minimum
+    is not recommended, as it will lead to a higher workload, because of the high forgetting rate.
 deck-config-please-save-your-changes-first = Please save your changes first.
 deck-config-workload-factor-change = Approximate workload: {$factor}x
     (compared to {$previousDR}% desired retention)
@@ -478,11 +484,20 @@ deck-config-percent-of-reviews =
        *[other] { $pct }% of { $reviews } reviews
     }
 deck-config-percent-input = { $pct }%
-# This message appears during FSRS parameter optimization.
 deck-config-checking-for-improvement = Checking for improvement...
 deck-config-optimizing-preset = Optimizing preset { $current_count }/{ $total_count }...
 deck-config-fsrs-must-be-enabled = FSRS must be enabled first.
 deck-config-fsrs-params-optimal = The FSRS parameters currently appear to be optimal.
+deck-config-fsrs-bad-fit-warning = Health Check: 
+    Your memory is difficult for FSRS to predict. Recommendations:
+
+    - Suspend or reformulate leeches.
+    - Use the answer buttons consistently. Keep in mind that "Hard" is a passing grade, not a failing grade.
+    - Understand before you memorize.
+
+    If you follow these suggestions, performance will usually improve over the next few months.
+deck-config-fsrs-good-fit = Health Check: 
+    FSRS can adapt to your memory well.
 
 deck-config-fsrs-params-no-reviews = No reviews found. Make sure this preset is assigned to all decks (including subdecks) that you want to optimize, and try again.
 
@@ -496,10 +511,7 @@ deck-config-desired-retention-below-optimal = Your desired retention is below op
 # Description of the y axis in the FSRS simulation
 # diagram (Deck options -> FSRS) showing the total number of
 # cards that can be recalled or retrieved on a specific date.
-deck-config-fsrs-simulator-experimental = FSRS Simulator (Experimental)
+deck-config-fsrs-simulator-experimental = FSRS simulator (experimental)
-deck-config-fsrs-simulate-desired-retention-experimental = FSRS Desired Retention Simulator (Experimental)
-deck-config-fsrs-simulate-save-preset = After optimizing, please save your deck preset before running the simulator.
-deck-config-fsrs-desired-retention-help-me-decide-experimental = Help Me Decide (Experimental)
 deck-config-additional-new-cards-to-simulate = Additional new cards to simulate
 deck-config-simulate = Simulate
 deck-config-clear-last-simulate = Clear Last Simulation
@@ -508,45 +520,13 @@ deck-config-advanced-settings = Advanced Settings
 deck-config-smooth-graph = Smooth graph
 deck-config-suspend-leeches = Suspend leeches
 deck-config-save-options-to-preset = Save Changes to Preset
-deck-config-save-options-to-preset-confirm = Overwrite the options in your current preset with the options that are currently set in the simulator?
 # Radio button in the FSRS simulation diagram (Deck options -> FSRS) selecting
 # to show the total number of cards that can be recalled or retrieved on a
 # specific date.
 deck-config-fsrs-simulator-radio-memorized = Memorized
-deck-config-fsrs-simulator-radio-ratio = Time / Memorized Ratio
-# $time here is pre-formatted e.g. "10 Seconds" 
-deck-config-fsrs-simulator-ratio-tooltip = { $time } per memorized card
-
-## Messages related to the FSRS scheduler’s health check. The health check determines whether the correlation between FSRS predictions and your memory is good or bad. It can be optionally triggered as part of the "Optimize" function.
-
-# Checkbox
-deck-config-health-check = Check health when optimizing
-# Message box showing the result of the health check
-deck-config-fsrs-bad-fit-warning = Health Check:
-    Your memory is difficult for FSRS to predict. Recommendations:
-
-    - Suspend or reformulate any cards you constantly forget.
-    - Use the answer buttons consistently. Keep in mind that "Hard" is a passing grade, not a failing grade.
-    - Understand before you memorize.
-
-    If you follow these suggestions, performance will usually improve over the next few months.
-# Message box showing the result of the health check
-deck-config-fsrs-good-fit = Health Check:
-    FSRS can adapt to your memory well.
 
 ## NO NEED TO TRANSLATE. This text is no longer used by Anki, and will be removed in the future.
 
-deck-config-unable-to-determine-desired-retention =
-    Unable to determine a minimum recommended retention.
-deck-config-predicted-minimum-recommended-retention = Minimum recommended retention: { $num }
-deck-config-compute-minimum-recommended-retention = Minimum recommended retention
-deck-config-compute-optimal-retention-tooltip4 =
-    This tool will attempt to find the desired retention value 
-    that will lead to the most material learnt, in the least amount of time. The calculated number can serve as a reference
-    when deciding what to set your desired retention to. You may wish to choose a higher desired retention if you’re 
-    willing to invest more study time to achieve it. Setting your desired retention lower than the minimum
-    is not recommended, as it will lead to a higher workload, because of the high forgetting rate.
-deck-config-plotted-on-x-axis = (Plotted on the X-axis)
 deck-config-a-100-day-interval = 
     { $days ->
         [one] A 100 day interval will become { $days } day.

ftl/core/importing.ftl

@@ -48,7 +48,6 @@ importing-merge-notetypes-help =
     Warning: This will require a one-way sync, and may mark existing notes as modified.
 importing-mnemosyne-20-deck-db = Mnemosyne 2.0 Deck (*.db)
 importing-multicharacter-separators-are-not-supported-please = Multi-character separators are not supported. Please enter one character only.
-importing-new-deck-will-be-created = A new deck will be created: { $name }
 importing-notes-added-from-file = Notes added from file: { $val }
 importing-notes-found-in-file = Notes found in file: { $val }
 importing-notes-skipped-as-theyre-already-in = Notes skipped, as up-to-date copies are already in your collection: { $val }
@@ -66,6 +65,7 @@ importing-with-deck-configs-help =
     If enabled, any deck options that the deck sharer included will also be imported.
     Otherwise, all decks will be assigned the default preset.
 importing-packaged-anki-deckcollection-apkg-colpkg-zip = Packaged Anki Deck/Collection (*.apkg *.colpkg *.zip)
+importing-pauker-18-lesson-paugz = Pauker 1.8 Lesson (*.pau.gz)
 # the '|' character
 importing-pipe = Pipe
 # Warning displayed when the csv import preview table is clipped (some columns were hidden)
@@ -78,6 +78,7 @@ importing-rows-had-num1d-fields-expected-num2d = '{ $row }' had { $found } field
 importing-selected-file-was-not-in-utf8 = Selected file was not in UTF-8 format. Please see the importing section of the manual.
 importing-semicolon = Semicolon
 importing-skipped = Skipped
+importing-supermemo-xml-export-xml = Supermemo XML export (*.xml)
 importing-tab = Tab
 importing-tag-modified-notes = Tag modified notes:
 importing-text-separated-by-tabs-or-semicolons = Text separated by tabs or semicolons (*)
@@ -251,5 +252,3 @@ importing-importing-collection = Importing collection...
 importing-unable-to-import-filename = Unable to import { $filename }: file type not supported
 importing-notes-that-could-not-be-imported = Notes that could not be imported as note type has changed: { $val }
 importing-added = Added
-importing-pauker-18-lesson-paugz = Pauker 1.8 Lesson (*.pau.gz)
-importing-supermemo-xml-export-xml = Supermemo XML export (*.xml)

ftl/core/preferences.ftl

@@ -34,7 +34,7 @@ preferences-when-adding-default-to-current-deck = When adding, default to curren
 preferences-you-can-restore-backups-via-fileswitch = You can restore backups via File > Switch Profile.
 preferences-legacy-timezone-handling = Legacy timezone handling (buggy, but required for AnkiDroid <= 2.14)
 preferences-default-search-text = Default search text
-preferences-default-search-text-example = e.g. "deck:current"
+preferences-default-search-text-example = eg. 'deck:current '
 preferences-theme = Theme
 preferences-theme-follow-system = Follow System
 preferences-theme-light = Light

ftl/core/statistics.ftl

@@ -80,7 +80,7 @@ statistics-reviews =
 # This fragment of the tooltip in the FSRS simulation
 # diagram (Deck options -> FSRS) shows the total number of
 # cards that can be recalled or retrieved on a specific date.
-statistics-memorized = {$memorized} cards memorized
+statistics-memorized = {$memorized} memorized
 statistics-today-title = Today
 statistics-today-again-count = Again count:
 statistics-today-type-counts = Learn: { $learnCount }, Review: { $reviewCount }, Relearn: { $relearnCount }, Filtered: { $filteredCount }
@@ -99,9 +99,9 @@ statistics-counts-relearning-cards = Relearning
 statistics-counts-title = Card Counts
 statistics-counts-separate-suspended-buried-cards = Separate suspended/buried cards
 
-## Retention represents your actual retention from past reviews, in
+## True Retention represents your actual retention rate from past reviews, in
-## comparison to the "desired retention" setting of FSRS, which forecasts
+## comparison to the "desired retention" parameter of FSRS, which forecasts
-## future retention. Retention is the percentage of all reviewed cards
+## future retention. True Retention is the percentage of all reviewed cards
 ## that were marked as "Hard," "Good," or "Easy" within a specific time period.
 ##
 ## Most of these strings are used as column / row headings in a table.
@@ -112,9 +112,9 @@ statistics-counts-separate-suspended-buried-cards = Separate suspended/buried ca
 ## N.B. Stats cards may be very small on mobile devices and when the Stats
 ##      window is certain sizes.
 
-statistics-true-retention-title = Retention
+statistics-true-retention-title = True Retention
 statistics-true-retention-subtitle = Pass rate of cards with an interval ≥ 1 day.
-statistics-true-retention-tooltip = If you are using FSRS, your retention is expected to be close to your desired retention. Please keep in mind that data for a single day is noisy, so it's better to look at monthly data.
+statistics-true-retention-tooltip = If you are using FSRS, your true retention is expected to be close to your desired retention. Please keep in mind that data for a single day is noisy, so it's better to look at monthly data.
 statistics-true-retention-range = Range
 statistics-true-retention-pass = Pass
 statistics-true-retention-fail = Fail

ftl/core/studying.ftl

@@ -46,20 +46,6 @@ studying-type-answer-unknown-field = Type answer: unknown field { $val }
 studying-unbury = Unbury
 studying-what-would-you-like-to-unbury = What would you like to unbury?
 studying-you-havent-recorded-your-voice-yet = You haven't recorded your voice yet.
-studying-card-studied-in-minute =
-    { $cards ->
-        [one] { $cards } card
-       *[other] { $cards } cards
-    } studied in
-    { $minutes ->
-        [one] { $minutes } minute.
-       *[other] { $minutes } minutes.
-    }
-studying-question-time-elapsed = Question time elapsed
-studying-answer-time-elapsed = Answer time elapsed
-
-## OBSOLETE; you do not need to translate this
-
 studying-card-studied-in =
     { $count ->
         [one] { $count } card studied in
@@ -70,3 +56,5 @@ studying-minute =
         [one] { $count } minute.
        *[other] { $count } minutes.
     }
+studying-question-time-elapsed = Question time elapsed
+studying-answer-time-elapsed = Answer time elapsed

ftl/qt-repo

@@ -1 +1 @@
-Subproject commit fd5f984785ad07a0d3dbd893ee3d7e3671eaebd6
+Subproject commit fbe9d1c731f7ad09953e63fdb0c455a6d3a3b6be

ftl/qt/qt-accel.ftl

@@ -1,5 +1,4 @@
 qt-accel-about = &About
-qt-accel-about-mac = About Anki...
 qt-accel-cards = &Cards
 qt-accel-check-database = &Check Database
 qt-accel-check-media = Check &Media
@@ -46,4 +45,3 @@ qt-accel-zoom-editor-in = Zoom Editor &In
 qt-accel-zoom-editor-out = Zoom Editor &Out
 qt-accel-create-backup = Create &Backup
 qt-accel-load-backup = &Revert to Backup
-qt-accel-upgrade-downgrade = Upgrade/Downgrade

ftl/qt/qt-misc.ftl

@@ -73,7 +73,6 @@ qt-misc-second =
 qt-misc-layout-auto-enabled = Responsive layout enabled
 qt-misc-layout-vertical-enabled = Vertical layout enabled
 qt-misc-layout-horizontal-enabled = Horizontal layout enabled
-qt-misc-open-anki-launcher = Change to a different Anki version?
 
 ## deprecated- these strings will be removed in the future, and do not need
 ## to be translated

ftl/src/serialize.rs

@@ -435,7 +435,7 @@ impl TextWriter {
             item = item.trim_start_matches(' ');
         }
 
-        write!(self.buffer, "{item}")
+        write!(self.buffer, "{}", item)
     }
 
     fn write_char_into_indent(&mut self, ch: char) {

ftl/src/string/mod.rs

@@ -67,7 +67,7 @@ fn additional_template_folder(dst_folder: &Utf8Path) -> Option<Utf8PathBuf> {
 
 fn all_langs(lang_folder: &Utf8Path) -> Result<Vec<Utf8PathBuf>> {
     std::fs::read_dir(lang_folder)
-        .with_context(|| format!("reading {lang_folder:?}"))?
+        .with_context(|| format!("reading {:?}", lang_folder))?
         .filter_map(Result::ok)
         .map(|e| Ok(e.path().utf8()?))
         .collect()

package.json

@@ -19,8 +19,8 @@
         "@poppanator/sveltekit-svg": "^5.0.0",
         "@sqltools/formatter": "^1.2.2",
         "@sveltejs/adapter-static": "^3.0.0",
-        "@sveltejs/kit": "^2.22.2",
+        "@sveltejs/kit": "^2.20.7",
-        "@sveltejs/vite-plugin-svelte": "5.1",
+        "@sveltejs/vite-plugin-svelte": "4.0.0",
         "@types/bootstrap": "^5.0.12",
         "@types/codemirror": "^5.60.0",
         "@types/d3": "^7.0.0",
@@ -30,7 +30,7 @@
         "@types/jqueryui": "^1.12.13",
         "@types/lodash-es": "^4.17.4",
         "@types/marked": "^5.0.0",
-        "@types/node": "^22",
+        "@types/node": "^20",
         "@typescript-eslint/eslint-plugin": "^5.60.1",
         "@typescript-eslint/parser": "^5.60.1",
         "caniuse-lite": "^1.0.30001431",
@@ -48,16 +48,16 @@
         "prettier": "^3.4.2",
         "prettier-plugin-svelte": "^3.3.2",
         "sass": "<1.77",
-        "svelte": "^5.34.9",
+        "svelte": "^5.17.3",
-        "svelte-check": "^4.2.2",
+        "svelte-check": "^3.4.4",
-        "svelte-preprocess": "^6.0.3",
+        "svelte-preprocess": "^5.0.4",
         "svelte-preprocess-esbuild": "^3.0.1",
         "svgo": "^3.2.0",
         "tslib": "^2.0.3",
-        "tsx": "^4.8.1",
+        "tsx": "^3.12.0",
         "typescript": "^5.0.4",
-        "vite": "6",
+        "vite": "5.4.19",
-        "vitest": "^3"
+        "vitest": "^2"
     },
     "dependencies": {
         "@bufbuild/protobuf": "^1.2.1",
@@ -81,9 +81,7 @@
     },
     "resolutions": {
         "canvas": "npm:empty-npm-package@1.0.0",
-        "cookie": "0.7.0",
+        "cookie": "0.7.0"
-        "devalue": "^5.3.2",
-        "vite": "6"
     },
     "browserslist": [
         "defaults",

proto/anki/cards.proto

@@ -51,7 +51,6 @@ message Card {
   optional FsrsMemoryState memory_state = 20;
   optional float desired_retention = 21;
   optional float decay = 22;
-  optional int64 last_review_time_secs = 23;
   string custom_data = 19;
 }
 

proto/anki/collection.proto

@@ -20,7 +20,6 @@ service CollectionService {
   rpc LatestProgress(generic.Empty) returns (Progress);
   rpc SetWantsAbort(generic.Empty) returns (generic.Empty);
   rpc SetLoadBalancerEnabled(generic.Bool) returns (OpChanges);
-  rpc GetCustomColours(generic.Empty) returns (GetCustomColoursResponse);
 }
 
 // Implicitly includes any of the above methods that are not listed in the
@@ -164,7 +163,3 @@ message CreateBackupRequest {
   bool force = 2;
   bool wait_for_completion = 3;
 }
-
-message GetCustomColoursResponse {
-  repeated string colours = 1;
-}

proto/anki/config.proto

@@ -56,7 +56,6 @@ message ConfigKey {
     RENDER_LATEX = 25;
     LOAD_BALANCER_ENABLED = 26;
     FSRS_SHORT_TERM_WITH_STEPS_ENABLED = 27;
-    FSRS_LEGACY_EVALUATE = 28;
   }
   enum String {
     SET_DUE_BROWSER = 0;

proto/anki/deck_config.proto

@@ -40,10 +40,12 @@ message DeckConfigId {
 message GetRetentionWorkloadRequest {
   repeated float w = 1;
   string search = 2;
+  float before = 3;
+  float after = 4;
 }
 
 message GetRetentionWorkloadResponse {
-  map<uint32, float> costs = 1;
+  float factor = 1;
 }
 
 message GetIgnoredBeforeCountRequest {
@@ -217,8 +219,6 @@ message DeckConfigsForUpdate {
       bool review_today_active = 5;
       // Whether new_today applies to today or a past day.
       bool new_today_active = 6;
-      // Deck-specific desired retention override
-      optional float desired_retention = 7;
     }
     string name = 1;
     int64 config_id = 2;
@@ -236,7 +236,6 @@ message DeckConfigsForUpdate {
   bool new_cards_ignore_review_limit = 7;
   bool fsrs = 8;
   bool fsrs_health_check = 11;
-  bool fsrs_legacy_evaluate = 12;
   bool apply_all_parent_limits = 9;
   uint32 days_since_last_fsrs_optimize = 10;
 }

proto/anki/decks.proto

@@ -83,8 +83,6 @@ message Deck {
     optional uint32 new_limit = 7;
     DayLimit review_limit_today = 8;
     DayLimit new_limit_today = 9;
-    // Deck-specific desired retention override
-    optional float desired_retention = 10;
 
     reserved 12 to 15;
   }

proto/anki/frontend.proto

@@ -27,9 +27,6 @@ service FrontendService {
   rpc deckOptionsRequireClose(generic.Empty) returns (generic.Empty);
   // Warns python that the deck option web view is ready to receive requests.
   rpc deckOptionsReady(generic.Empty) returns (generic.Empty);
-
-  // Save colour picker's custom colour palette
-  rpc SaveCustomColours(generic.Empty) returns (generic.Empty);
 }
 
 service BackendFrontendService {}

proto/anki/import_export.proto

@@ -176,12 +176,9 @@ message CsvMetadata {
   // to determine the number of columns.
   repeated string column_labels = 5;
   oneof deck {
-    // id of an existing deck
     int64 deck_id = 6;
     // One-based. 0 means n/a.
     uint32 deck_column = 7;
-    // name of new deck to be created
-    string deck_name = 17;
   }
   oneof notetype {
     // One notetype for all rows with given column mapping.

proto/anki/notes.proto

@@ -59,7 +59,7 @@ message AddNoteRequest {
 }
 
 message AddNoteResponse {
-  collection.OpChangesWithCount changes = 1;
+  collection.OpChanges changes = 1;
   int64 note_id = 2;
 }
 

proto/anki/scheduler.proto

@@ -55,11 +55,7 @@ service SchedulerService {
       returns (ComputeOptimalRetentionResponse);
   rpc SimulateFsrsReview(SimulateFsrsReviewRequest)
       returns (SimulateFsrsReviewResponse);
-  rpc SimulateFsrsWorkload(SimulateFsrsReviewRequest)
-      returns (SimulateFsrsWorkloadResponse);
   rpc EvaluateParams(EvaluateParamsRequest) returns (EvaluateParamsResponse);
-  rpc EvaluateParamsLegacy(EvaluateParamsLegacyRequest)
-      returns (EvaluateParamsResponse);
   rpc ComputeMemoryState(cards.CardId) returns (ComputeMemoryStateResponse);
   // The number of days the calculated interval was fuzzed by on the previous
   // review (if any). Utilized by the FSRS add-on.
@@ -406,9 +402,6 @@ message SimulateFsrsReviewRequest {
   repeated float easy_days_percentages = 10;
   deck_config.DeckConfig.Config.ReviewCardOrder review_order = 11;
   optional uint32 suspend_after_lapse_count = 12;
-  float historical_retention = 13;
-  uint32 learning_step_count = 14;
-  uint32 relearning_step_count = 15;
 }
 
 message SimulateFsrsReviewResponse {
@@ -418,12 +411,6 @@ message SimulateFsrsReviewResponse {
   repeated float daily_time_cost = 4;
 }
 
-message SimulateFsrsWorkloadResponse {
-  map<uint32, float> cost = 1;
-  map<uint32, float> memorized = 2;
-  map<uint32, uint32> review_count = 3;
-}
-
 message ComputeOptimalRetentionResponse {
   float optimal_retention = 1;
 }
@@ -455,12 +442,6 @@ message EvaluateParamsRequest {
   uint32 num_of_relearning_steps = 3;
 }
 
-message EvaluateParamsLegacyRequest {
-  repeated float params = 1;
-  string search = 2;
-  int64 ignore_revlogs_before_ms = 3;
-}
-
 message EvaluateParamsResponse {
   float log_loss = 1;
   float rmse_bins = 2;
@@ -469,7 +450,6 @@ message EvaluateParamsResponse {
 message ComputeMemoryStateResponse {
   optional cards.FsrsMemoryState state = 1;
   float desired_retention = 2;
-  float decay = 3;
 }
 
 message FuzzDeltaRequest {

proto/anki/search.proto

@@ -74,15 +74,10 @@ message SearchNode {
     repeated SearchNode nodes = 1;
     Joiner joiner = 2;
   }
-  enum FieldSearchMode {
-    FIELD_SEARCH_MODE_NORMAL = 0;
-    FIELD_SEARCH_MODE_REGEX = 1;
-    FIELD_SEARCH_MODE_NOCOMBINING = 2;
-  }
   message Field {
     string field_name = 1;
     string text = 2;
-    FieldSearchMode mode = 3;
+    bool is_re = 3;
   }
 
   oneof filter {

pylib/anki/_backend.py

@@ -46,6 +46,7 @@ from .errors import (
 
 # the following comment is required to suppress a warning that only shows up
 # when there are other pylint failures
+# pylint: disable=c-extension-no-member
 if _rsbridge.buildhash() != anki.buildinfo.buildhash:
     raise Exception(
         f"""rsbridge and anki build hashes do not match:
@@ -163,7 +164,7 @@ class RustBackend(RustBackendGenerated):
         finally:
             elapsed = time.time() - start
             if current_thread() is main_thread() and elapsed > 0.2:
-                print(f"blocked main thread for {int(elapsed * 1000)}ms:")
+                print(f"blocked main thread for {int(elapsed*1000)}ms:")
                 print("".join(traceback.format_stack()))
 
         err = backend_pb2.BackendError()
@@ -246,7 +247,7 @@ def backend_exception_to_pylib(err: backend_pb2.BackendError) -> Exception:
         return BackendError(err.message, help_page, context, backtrace)
 
     elif val == kind.SEARCH_ERROR:
-        return SearchError(err.message, help_page, context, backtrace)
+        return SearchError(markdown(err.message), help_page, context, backtrace)
 
     elif val == kind.UNDO_EMPTY:
         return UndoEmpty(err.message, help_page, context, backtrace)

pylib/anki/cards.py

@@ -7,7 +7,7 @@ import pprint
 import time
 from typing import NewType
 
-import anki
+import anki  # pylint: disable=unused-import
 import anki.collection
 import anki.decks
 import anki.notes
@@ -49,7 +49,6 @@ class Card(DeprecatedNamesMixin):
     memory_state: FSRSMemoryState | None
     desired_retention: float | None
     decay: float | None
-    last_review_time: int | None
 
     def __init__(
         self,
@@ -104,11 +103,6 @@ class Card(DeprecatedNamesMixin):
             card.desired_retention if card.HasField("desired_retention") else None
         )
         self.decay = card.decay if card.HasField("decay") else None
-        self.last_review_time = (
-            card.last_review_time_secs
-            if card.HasField("last_review_time_secs")
-            else None
-        )
 
     def _to_backend_card(self) -> cards_pb2.Card:
         # mtime & usn are set by backend
@@ -133,7 +127,6 @@ class Card(DeprecatedNamesMixin):
             memory_state=self.memory_state,
             desired_retention=self.desired_retention,
             decay=self.decay,
-            last_review_time_secs=self.last_review_time,
         )
 
     @deprecated(info="please use col.update_card()")

pylib/anki/collection.py

@@ -122,7 +122,6 @@ class ComputedMemoryState:
     desired_retention: float
     stability: float | None = None
     difficulty: float | None = None
-    decay: float | None = None
 
 
 @dataclass
@@ -158,7 +157,7 @@ class Collection(DeprecatedNamesMixin):
         self.tags = TagManager(self)
         self.conf = ConfigManager(self)
         self._load_scheduler()
-        self._startReps = 0
+        self._startReps = 0  # pylint: disable=invalid-name
 
     def name(self) -> Any:
         return os.path.splitext(os.path.basename(self.path))[0]
@@ -511,7 +510,9 @@ class Collection(DeprecatedNamesMixin):
     # Utils
     ##########################################################################
 
-    def nextID(self, type: str, inc: bool = True) -> Any:
+    def nextID(  # pylint: disable=invalid-name
+        self, type: str, inc: bool = True
+    ) -> Any:
         type = f"next{type.capitalize()}"
         id = self.conf.get(type, 1)
         if inc:
@@ -528,7 +529,7 @@ class Collection(DeprecatedNamesMixin):
     def new_note(self, notetype: NotetypeDict) -> Note:
         return Note(self, notetype)
 
-    def add_note(self, note: Note, deck_id: DeckId) -> OpChangesWithCount:
+    def add_note(self, note: Note, deck_id: DeckId) -> OpChanges:
         hooks.note_will_be_added(self, note, deck_id)
         out = self._backend.add_note(note=note._to_backend_note(), deck_id=deck_id)
         note.id = NoteId(out.note_id)
@@ -847,6 +848,7 @@ class Collection(DeprecatedNamesMixin):
         )
 
     def _pb_search_separator(self, operator: SearchJoiner) -> SearchNode.Group.Joiner.V:
+        # pylint: disable=no-member
         if operator == "AND":
             return SearchNode.Group.Joiner.AND
         else:
@@ -864,9 +866,7 @@ class Collection(DeprecatedNamesMixin):
                 return column
         return None
 
-    def browser_row_for_id(
+    def browser_row_for_id(self, id_: int) -> tuple[
-        self, id_: int
-    ) -> tuple[
         Generator[tuple[str, bool, BrowserRow.Cell.TextElideMode.V], None, None],
         BrowserRow.Color.V,
         str,
@@ -1189,13 +1189,9 @@ class Collection(DeprecatedNamesMixin):
                 desired_retention=resp.desired_retention,
                 stability=resp.state.stability,
                 difficulty=resp.state.difficulty,
-                decay=resp.decay,
             )
         else:
-            return ComputedMemoryState(
+            return ComputedMemoryState(desired_retention=resp.desired_retention)
-                desired_retention=resp.desired_retention,
-                decay=resp.decay,
-            )
 
     def fuzz_delta(self, card_id: CardId, interval: int) -> int:
         "The delta days of fuzz applied if reviewing the card in v3."
@@ -1211,6 +1207,8 @@ class Collection(DeprecatedNamesMixin):
     # the count on things like edits, which we probably could do by checking
     # the previous state in moveToState.
 
+    # pylint: disable=invalid-name
+
     def startTimebox(self) -> None:
         self._startTime = time.time()
         self._startReps = self.sched.reps

pylib/anki/exporting.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 from __future__ import annotations
 
@@ -350,7 +351,7 @@ class AnkiPackageExporter(AnkiExporter):
         colfile = path.replace(".apkg", ".anki2")
         AnkiExporter.exportInto(self, colfile)
         # prevent older clients from accessing
-
+        # pylint: disable=unreachable
         self._addDummyCollection(z)
         z.write(colfile, "collection.anki21")
 

pylib/anki/find.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 from __future__ import annotations
 

pylib/anki/foreign_data/__init__.py

@@ -1,7 +1,8 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
-"""Helpers for serializing third-party collections to a common JSON form."""
+"""Helpers for serializing third-party collections to a common JSON form.
+"""
 
 from __future__ import annotations
 

pylib/anki/foreign_data/mnemosyne.py

@@ -175,8 +175,8 @@ class MnemoFact:
     def fact_view(self) -> type[MnemoFactView]:
         try:
             fact_view = self.cards[0].fact_view_id
-        except IndexError:
+        except IndexError as err:
-            return FrontOnly
+            raise Exception(f"Fact {id} has no cards") from err
 
         if fact_view.startswith("1.") or fact_view.startswith("1::"):
             return FrontOnly
@@ -187,7 +187,7 @@ class MnemoFact:
         elif fact_view.startswith("5.1"):
             return Cloze
 
-        raise Exception(f"Fact {self.id} has unknown fact view: {fact_view}")
+        raise Exception(f"Fact {id} has unknown fact view: {fact_view}")
 
     def anki_fields(self, fact_view: type[MnemoFactView]) -> list[str]:
         return [munge_field(self.fields.get(k, "")) for k in fact_view.field_keys]

pylib/anki/hooks.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 """
 Tools for extending Anki.

pylib/anki/importing/__init__.py

@@ -11,6 +11,8 @@ from anki.importing.apkg import AnkiPackageImporter
 from anki.importing.base import Importer
 from anki.importing.csvfile import TextImporter
 from anki.importing.mnemo import MnemosyneImporter
+from anki.importing.pauker import PaukerImporter
+from anki.importing.supermemo_xml import SupermemoXmlImporter  # type: ignore
 from anki.lang import TR
 
 
@@ -22,6 +24,8 @@ def importers(col: Collection) -> Sequence[tuple[str, type[Importer]]]:
             AnkiPackageImporter,
         ),
         (col.tr.importing_mnemosyne_20_deck_db(), MnemosyneImporter),
+        (col.tr.importing_supermemo_xml_export_xml(), SupermemoXmlImporter),
+        (col.tr.importing_pauker_18_lesson_paugz(), PaukerImporter),
     ]
     anki.hooks.importing_importers(importers)
     return importers

pylib/anki/importing/anki2.py

@@ -1,7 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
-
+# pylint: disable=invalid-name
 from __future__ import annotations
 
 import os

pylib/anki/importing/apkg.py

@@ -1,7 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
-
+# pylint: disable=invalid-name
 from __future__ import annotations
 
 import json

pylib/anki/importing/base.py

@@ -1,7 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
-
+# pylint: disable=invalid-name
 from __future__ import annotations
 
 from typing import Any

pylib/anki/importing/csvfile.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 from __future__ import annotations
 
@@ -143,6 +144,7 @@ class TextImporter(NoteImporter):
         self.close()
         zuper = super()
         if hasattr(zuper, "__del__"):
+            # pylint: disable=no-member
             zuper.__del__(self)  # type: ignore
 
     def noteFromFields(self, fields: list[str]) -> ForeignNote:

pylib/anki/importing/mnemo.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 import re
 import time
@@ -34,6 +35,7 @@ f._id=d._fact_id"""
         ):
             if id != curid:
                 if note:
+                    # pylint: disable=unsubscriptable-object
                     notes[note["_id"]] = note
                 note = {"_id": _id}
                 curid = id
@@ -183,6 +185,7 @@ acq_reps+ret_reps, lapses, card_type_id from cards"""
             state = dict(n=1)
 
             def repl(match):
+                # pylint: disable=cell-var-from-loop
                 # replace [...] with cloze refs
                 res = "{{c%d::%s}}" % (state["n"], match.group(1))
                 state["n"] += 1

pylib/anki/importing/noteimp.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 from __future__ import annotations
 
@@ -166,9 +167,9 @@ class NoteImporter(Importer):
             firsts[fld0] = True
             # already exists?
             found = False
-            if csum in csums:  # type: ignore[comparison-overlap]
+            if csum in csums:
                 # csum is not a guarantee; have to check
-                for id in csums[csum]:  # type: ignore[index]
+                for id in csums[csum]:
                     flds = self.col.db.scalar("select flds from notes where id = ?", id)
                     sflds = split_fields(flds)
                     if fld0 == sflds[0]:

pylib/anki/importing/pauker.py

@@ -0,0 +1,94 @@
+# Copyright: Andreas Klauer <Andreas.Klauer@metamorpher.de>
+# License: BSD-3
+
+# pylint: disable=invalid-name
+
+import gzip
+import html
+import math
+import random
+import time
+import xml.etree.ElementTree as ET
+
+from anki.importing.noteimp import ForeignCard, ForeignNote, NoteImporter
+from anki.stdmodels import _legacy_add_forward_reverse
+
+ONE_DAY = 60 * 60 * 24
+
+
+class PaukerImporter(NoteImporter):
+    """Import Pauker 1.8 Lesson (*.pau.gz)"""
+
+    needMapper = False
+    allowHTML = True
+
+    def run(self):
+        model = _legacy_add_forward_reverse(self.col)
+        model["name"] = "Pauker"
+        self.col.models.save(model, updateReqs=False)
+        self.col.models.set_current(model)
+        self.model = model
+        self.initMapping()
+        NoteImporter.run(self)
+
+    def fields(self):
+        """Pauker is Front/Back"""
+        return 2
+
+    def foreignNotes(self):
+        """Build and return a list of notes."""
+        notes = []
+
+        try:
+            f = gzip.open(self.file)
+            tree = ET.parse(f)  # type: ignore
+            lesson = tree.getroot()
+            assert lesson.tag == "Lesson"
+        finally:
+            f.close()
+
+        index = -4
+
+        for batch in lesson.findall("./Batch"):
+            index += 1
+
+            for card in batch.findall("./Card"):
+                # Create a note for this card.
+                front = card.findtext("./FrontSide/Text")
+                back = card.findtext("./ReverseSide/Text")
+                note = ForeignNote()
+                assert front and back
+                note.fields = [
+                    html.escape(x.strip())
+                    .replace("\n", "<br>")
+                    .replace("  ", " &nbsp;")
+                    for x in [front, back]
+                ]
+                notes.append(note)
+
+                # Determine due date for cards.
+                frontdue = card.find("./FrontSide[@LearnedTimestamp]")
+                backdue = card.find("./ReverseSide[@Batch][@LearnedTimestamp]")
+
+                if frontdue is not None:
+                    note.cards[0] = self._learnedCard(
+                        index, int(frontdue.attrib["LearnedTimestamp"])
+                    )
+
+                if backdue is not None:
+                    note.cards[1] = self._learnedCard(
+                        int(backdue.attrib["Batch"]),
+                        int(backdue.attrib["LearnedTimestamp"]),
+                    )
+
+        return notes
+
+    def _learnedCard(self, batch, timestamp):
+        ivl = math.exp(batch)
+        now = time.time()
+        due = ivl - (now - timestamp / 1000.0) / ONE_DAY
+        fc = ForeignCard()
+        fc.due = self.col.sched.today + int(due + 0.5)
+        fc.ivl = random.randint(int(ivl * 0.90), int(ivl + 0.5))
+        fc.factor = random.randint(1500, 2500)
+        return fc

pylib/anki/importing/supermemo_xml.py

@@ -0,0 +1,484 @@
+# Copyright: petr.michalec@gmail.com
+# License: GNU GPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
+# pytype: disable=attribute-error
+# type: ignore
+# pylint: disable=C
+from __future__ import annotations
+
+import re
+import sys
+import time
+import unicodedata
+from string import capwords
+from xml.dom import minidom
+from xml.dom.minidom import Element, Text
+
+from anki.collection import Collection
+from anki.importing.noteimp import ForeignCard, ForeignNote, NoteImporter
+from anki.stdmodels import _legacy_add_basic_model
+
+
+class SmartDict(dict):
+    """
+    See http://www.peterbe.com/plog/SmartDict
+    Copyright 2005, Peter Bengtsson, peter@fry-it.com
+    0BSD
+
+    A smart dict can be instantiated either from a pythonic dict
+    or an instance object (eg. SQL recordsets) but it ensures that you can
+    do all the convenient lookups such as x.first_name, x['first_name'] or
+    x.get('first_name').
+    """
+
+    def __init__(self, *a, **kw) -> None:
+        if a:
+            if isinstance(type(a[0]), dict):
+                kw.update(a[0])
+            elif isinstance(type(a[0]), object):
+                kw.update(a[0].__dict__)
+            elif hasattr(a[0], "__class__") and a[0].__class__.__name__ == "SmartDict":
+                kw.update(a[0].__dict__)
+
+        dict.__init__(self, **kw)
+        self.__dict__ = self
+
+
+class SuperMemoElement(SmartDict):
+    "SmartDict wrapper to store SM Element data"
+
+    def __init__(self, *a, **kw) -> None:
+        SmartDict.__init__(self, *a, **kw)
+        # default content
+        self.__dict__["lTitle"] = None
+        self.__dict__["Title"] = None
+        self.__dict__["Question"] = None
+        self.__dict__["Answer"] = None
+        self.__dict__["Count"] = None
+        self.__dict__["Type"] = None
+        self.__dict__["ID"] = None
+        self.__dict__["Interval"] = None
+        self.__dict__["Lapses"] = None
+        self.__dict__["Repetitions"] = None
+        self.__dict__["LastRepetiton"] = None
+        self.__dict__["AFactor"] = None
+        self.__dict__["UFactor"] = None
+
+
+# This is an AnkiImporter
+class SupermemoXmlImporter(NoteImporter):
+    needMapper = False
+    allowHTML = True
+
+    """
+    Supermemo XML export's to Anki parser.
+    Goes through a SM collection and fetch all elements.
+
+    My SM collection was a big mess where topics and items were mixed.
+    I was unable to parse my content in a regular way like for loop on
+    minidom.getElementsByTagName() etc. My collection had also an
+    limitation, topics were splited into branches with max 100 items
+    on each. Learning themes were in deep structure. I wanted to have
+    full title on each element to be stored in tags.
+
+    Code should be upgrade to support importing of SM2006 exports.
+    """
+
+    def __init__(self, col: Collection, file: str) -> None:
+        """Initialize internal variables.
+        Pameters to be exposed to GUI are stored in self.META"""
+        NoteImporter.__init__(self, col, file)
+        m = _legacy_add_basic_model(self.col)
+        m["name"] = "Supermemo"
+        self.col.models.save(m)
+        self.initMapping()
+
+        self.lines = None
+        self.numFields = int(2)
+
+        # SmXmlParse VARIABLES
+        self.xmldoc = None
+        self.pieces = []
+        self.cntBuf = []  # to store last parsed data
+        self.cntElm = []  # to store SM Elements data
+        self.cntCol = []  # to store SM Colections data
+
+        # store some meta info related to parse algorithm
+        # SmartDict works like dict / class wrapper
+        self.cntMeta = SmartDict()
+        self.cntMeta.popTitles = False
+        self.cntMeta.title = []
+
+        # META stores controls of import script, should be
+        # exposed to import dialog. These are default values.
+        self.META = SmartDict()
+        self.META.resetLearningData = False  # implemented
+        self.META.onlyMemorizedItems = False  # implemented
+        self.META.loggerLevel = 2  # implemented 0no,1info,2error,3debug
+        self.META.tagAllTopics = True
+        self.META.pathsToBeTagged = [
+            "English for beginners",
+            "Advanced English 97",
+            "Phrasal Verbs",
+        ]  # path patterns to be tagged - in gui entered like 'Advanced English 97|My Vocablary'
+        self.META.tagMemorizedItems = True  # implemented
+        self.META.logToStdOutput = False  # implemented
+
+        self.notes = []
+
+    ## TOOLS
+
+    def _fudgeText(self, text: str) -> str:
+        "Replace sm syntax to Anki syntax"
+        text = text.replace("\n\r", "<br>")
+        text = text.replace("\n", "<br>")
+        return text
+
+    def _unicode2ascii(self, str: str) -> str:
+        "Remove diacritic punctuation from strings (titles)"
+        return "".join(
+            [
+                c
+                for c in unicodedata.normalize("NFKD", str)
+                if not unicodedata.combining(c)
+            ]
+        )
+
+    def _decode_htmlescapes(self, html: str) -> str:
+        """Unescape HTML code."""
+        # In case of bad formatted html you can import MinimalSoup etc.. see BeautifulSoup source code
+        from bs4 import BeautifulSoup
+
+        # my sm2004 also ecaped & char in escaped sequences.
+        html = re.sub("&amp;", "&", html)
+
+        # https://anki.tenderapp.com/discussions/ankidesktop/39543-anki-is-replacing-the-character-by-when-i-exit-the-html-edit-mode-ctrlshiftx
+        if html.find(">") < 0:
+            return html
+
+        # unescaped solitary chars < or > that were ok for minidom confuse btfl soup
+        # html = re.sub(u'>',u'&gt;',html)
+        # html = re.sub(u'<',u'&lt;',html)
+
+        return str(BeautifulSoup(html, "html.parser"))
+
+    def _afactor2efactor(self, af: float) -> float:
+        # Adapted from <http://www.supermemo.com/beta/xml/xml-core.htm>
+
+        # Ranges for A-factors and E-factors
+        af_min = 1.2
+        af_max = 6.9
+        ef_min = 1.3
+        ef_max = 3.3
+
+        # Sanity checks for the A-factor
+        if af < af_min:
+            af = af_min
+        elif af > af_max:
+            af = af_max
+
+        # Scale af to the range 0..1
+        af_scaled = (af - af_min) / (af_max - af_min)
+        # Rescale to the interval ef_min..ef_max
+        ef = ef_min + af_scaled * (ef_max - ef_min)
+
+        return ef
+
+    ## DEFAULT IMPORTER METHODS
+
+    def foreignNotes(self) -> list[ForeignNote]:
+        # Load file and parse it by minidom
+        self.loadSource(self.file)
+
+        # Migrating content / time consuming part
+        # addItemToCards is called for each sm element
+        self.logger("Parsing started.")
+        self.parse()
+        self.logger("Parsing done.")
+
+        # Return imported cards
+        self.total = len(self.notes)
+        self.log.append("%d cards imported." % self.total)
+        return self.notes
+
+    def fields(self) -> int:
+        return 2
+
+    ## PARSER METHODS
+
+    def addItemToCards(self, item: SuperMemoElement) -> None:
+        "This method actually do conversion"
+
+        # new anki card
+        note = ForeignNote()
+
+        # clean Q and A
+        note.fields.append(self._fudgeText(self._decode_htmlescapes(item.Question)))
+        note.fields.append(self._fudgeText(self._decode_htmlescapes(item.Answer)))
+        note.tags = []
+
+        # pre-process scheduling data
+        # convert learning data
+        if (
+            not self.META.resetLearningData
+            and int(item.Interval) >= 1
+            and getattr(item, "LastRepetition", None)
+        ):
+            # migration of LearningData algorithm
+            tLastrep = time.mktime(time.strptime(item.LastRepetition, "%d.%m.%Y"))
+            tToday = time.time()
+            card = ForeignCard()
+            card.ivl = int(item.Interval)
+            card.lapses = int(item.Lapses)
+            card.reps = int(item.Repetitions) + int(item.Lapses)
+            nextDue = tLastrep + (float(item.Interval) * 86400.0)
+            remDays = int((nextDue - time.time()) / 86400)
+            card.due = self.col.sched.today + remDays
+            card.factor = int(
+                self._afactor2efactor(float(item.AFactor.replace(",", "."))) * 1000
+            )
+            note.cards[0] = card
+
+        # categories & tags
+        # it's worth to have every theme (tree structure of sm collection) stored in tags, but sometimes not
+        # you can deceide if you are going to tag all toppics or just that containing some pattern
+        tTaggTitle = False
+        for pattern in self.META.pathsToBeTagged:
+            if (
+                item.lTitle is not None
+                and pattern.lower() in " ".join(item.lTitle).lower()
+            ):
+                tTaggTitle = True
+                break
+        if tTaggTitle or self.META.tagAllTopics:
+            # normalize - remove diacritic punctuation from unicode chars to ascii
+            item.lTitle = [self._unicode2ascii(topic) for topic in item.lTitle]
+
+            # Transform xyz / aaa / bbb / ccc on Title path to Tag  xyzAaaBbbCcc
+            #  clean things like [999] or [111-2222] from title path, example: xyz / [1000-1200] zyx / xyz
+            #  clean whitespaces
+            #  set Capital letters for first char of the word
+            tmp = list(
+                {re.sub(r"(\[[0-9]+\])", " ", i).replace("_", " ") for i in item.lTitle}
+            )
+            tmp = list({re.sub(r"(\W)", " ", i) for i in tmp})
+            tmp = list({re.sub("^[0-9 ]+$", "", i) for i in tmp})
+            tmp = list({capwords(i).replace(" ", "") for i in tmp})
+            tags = [j[0].lower() + j[1:] for j in tmp if j.strip() != ""]
+
+            note.tags += tags
+
+            if self.META.tagMemorizedItems and int(item.Interval) > 0:
+                note.tags.append("Memorized")
+
+            self.logger("Element tags\t- " + repr(note.tags), level=3)
+
+        self.notes.append(note)
+
+    def logger(self, text: str, level: int = 1) -> None:
+        "Wrapper for Anki logger"
+
+        dLevels = {0: "", 1: "Info", 2: "Verbose", 3: "Debug"}
+        if level <= self.META.loggerLevel:
+            # self.deck.updateProgress(_(text))
+
+            if self.META.logToStdOutput:
+                print(
+                    self.__class__.__name__
+                    + " - "
+                    + dLevels[level].ljust(9)
+                    + " -\t"
+                    + text
+                )
+
+    # OPEN AND LOAD
+    def openAnything(self, source):
+        """Open any source / actually only opening of files is used
+        @return an open handle which must be closed after use, i.e., handle.close()"""
+
+        if source == "-":
+            return sys.stdin
+
+        # try to open with urllib (if source is http, ftp, or file URL)
+        import urllib.error
+        import urllib.parse
+        import urllib.request
+
+        try:
+            return urllib.request.urlopen(source)
+        except OSError:
+            pass
+
+        # try to open with native open function (if source is pathname)
+        try:
+            return open(source, encoding="utf8")
+        except OSError:
+            pass
+
+        # treat source as string
+        import io
+
+        return io.StringIO(str(source))
+
+    def loadSource(self, source: str) -> None:
+        """Load source file and parse with xml.dom.minidom"""
+        self.source = source
+        self.logger("Load started...")
+        sock = open(self.source, encoding="utf8")
+        self.xmldoc = minidom.parse(sock).documentElement
+        sock.close()
+        self.logger("Load done.")
+
+    # PARSE
+    def parse(self, node: Text | Element | None = None) -> None:
+        "Parse method - parses document elements"
+
+        if node is None and self.xmldoc is not None:
+            node = self.xmldoc
+
+        _method = "parse_%s" % node.__class__.__name__
+        if hasattr(self, _method):
+            parseMethod = getattr(self, _method)
+            parseMethod(node)
+        else:
+            self.logger("No handler for method %s" % _method, level=3)
+
+    def parse_Document(self, node):
+        "Parse XML document"
+
+        self.parse(node.documentElement)
+
+    def parse_Element(self, node: Element) -> None:
+        "Parse XML element"
+
+        _method = "do_%s" % node.tagName
+        if hasattr(self, _method):
+            handlerMethod = getattr(self, _method)
+            handlerMethod(node)
+        else:
+            self.logger("No handler for method %s" % _method, level=3)
+            # print traceback.print_exc()
+
+    def parse_Text(self, node: Text) -> None:
+        "Parse text inside elements. Text is stored into local buffer."
+
+        text = node.data
+        self.cntBuf.append(text)
+
+    # def parse_Comment(self, node):
+    #    """
+    #    Source can contain XML comments, but we ignore them
+    #    """
+    #    pass
+
+    # DO
+    def do_SuperMemoCollection(self, node: Element) -> None:
+        "Process SM Collection"
+
+        for child in node.childNodes:
+            self.parse(child)
+
+    def do_SuperMemoElement(self, node: Element) -> None:
+        "Process SM Element (Type - Title,Topics)"
+
+        self.logger("=" * 45, level=3)
+
+        self.cntElm.append(SuperMemoElement())
+        self.cntElm[-1]["lTitle"] = self.cntMeta["title"]
+
+        # parse all child elements
+        for child in node.childNodes:
+            self.parse(child)
+
+        # strip all saved strings, just for sure
+        for key in list(self.cntElm[-1].keys()):
+            if hasattr(self.cntElm[-1][key], "strip"):
+                self.cntElm[-1][key] = self.cntElm[-1][key].strip()
+
+        # pop current element
+        smel = self.cntElm.pop()
+
+        # Process cntElm if is valid Item (and not an Topic etc..)
+        # if smel.Lapses != None and smel.Interval != None and smel.Question != None and smel.Answer != None:
+        if smel.Title is None and smel.Question is not None and smel.Answer is not None:
+            if smel.Answer.strip() != "" and smel.Question.strip() != "":
+                # migrate only memorized otherway skip/continue
+                if self.META.onlyMemorizedItems and not (int(smel.Interval) > 0):
+                    self.logger("Element skipped  \t- not memorized ...", level=3)
+                else:
+                    # import sm element data to Anki
+                    self.addItemToCards(smel)
+                    self.logger("Import element \t- " + smel["Question"], level=3)
+
+                    # print element
+                    self.logger("-" * 45, level=3)
+                    for key in list(smel.keys()):
+                        self.logger(
+                            "\t{} {}".format((key + ":").ljust(15), smel[key]), level=3
+                        )
+            else:
+                self.logger("Element skipped  \t- no valid Q and A ...", level=3)
+
+        else:
+            # now we know that item was topic
+            # parsing of whole node is now finished
+
+            # test if it's really topic
+            if smel.Title is not None:
+                # remove topic from title list
+                t = self.cntMeta["title"].pop()
+                self.logger("End of topic \t- %s" % (t), level=2)
+
+    def do_Content(self, node: Element) -> None:
+        "Process SM element Content"
+
+        for child in node.childNodes:
+            if hasattr(child, "tagName") and child.firstChild is not None:
+                self.cntElm[-1][child.tagName] = child.firstChild.data
+
+    def do_LearningData(self, node: Element) -> None:
+        "Process SM element LearningData"
+
+        for child in node.childNodes:
+            if hasattr(child, "tagName") and child.firstChild is not None:
+                self.cntElm[-1][child.tagName] = child.firstChild.data
+
+    # It's being processed in do_Content now
+    # def do_Question(self, node):
+    #    for child in node.childNodes: self.parse(child)
+    #    self.cntElm[-1][node.tagName]=self.cntBuf.pop()
+
+    # It's being processed in do_Content now
+    # def do_Answer(self, node):
+    #    for child in node.childNodes: self.parse(child)
+    #    self.cntElm[-1][node.tagName]=self.cntBuf.pop()
+
+    def do_Title(self, node: Element) -> None:
+        "Process SM element Title"
+
+        t = self._decode_htmlescapes(node.firstChild.data)
+        self.cntElm[-1][node.tagName] = t
+        self.cntMeta["title"].append(t)
+        self.cntElm[-1]["lTitle"] = self.cntMeta["title"]
+        self.logger("Start of topic \t- " + " / ".join(self.cntMeta["title"]), level=2)
+
+    def do_Type(self, node: Element) -> None:
+        "Process SM element Type"
+
+        if len(self.cntBuf) >= 1:
+            self.cntElm[-1][node.tagName] = self.cntBuf.pop()
+
+
+# if __name__ == '__main__':
+
+# for testing you can start it standalone
+
+# file = u'/home/epcim/hg2g/dev/python/sm2anki/ADVENG2EXP.xxe.esc.zaloha_FINAL.xml'
+# file = u'/home/epcim/hg2g/dev/python/anki/libanki/tests/importing/supermemo/original_ENGLISHFORBEGGINERS_noOEM.xml'
+# file = u'/home/epcim/hg2g/dev/python/anki/libanki/tests/importing/supermemo/original_ENGLISHFORBEGGINERS_oem_1250.xml'
+# file = str(sys.argv[1])
+# impo = SupermemoXmlImporter(Deck(),file)
+# impo.foreignCards()
+
+# sys.exit(1)
+
+# vim: ts=4 sts=2 ft=python

pylib/anki/lang.py

@@ -18,7 +18,7 @@ from anki._legacy import DeprecatedNamesMixinForModule
 TR = anki._fluent.LegacyTranslationEnum
 FormatTimeSpan = _pb.FormatTimespanRequest
 
-# When adding new languages here, check lang_to_disk_lang() below
+
 langs = sorted(
     [
         ("Afrikaans", "af_ZA"),
@@ -38,7 +38,6 @@ langs = sorted(
         ("Italiano", "it_IT"),
         ("lo jbobau", "jbo_EN"),
         ("Lenga d'òc", "oc_FR"),
-        ("Қазақша", "kk_KZ"),
         ("Magyar", "hu_HU"),
         ("Nederlands", "nl_NL"),
         ("Norsk", "nb_NO"),
@@ -65,7 +64,6 @@ langs = sorted(
         ("Українська мова", "uk_UA"),
         ("Հայերեն", "hy_AM"),
         ("עִבְרִית", "he_IL"),
-        ("ייִדיש", "yi"),
         ("العربية", "ar_SA"),
         ("فارسی", "fa_IR"),
         ("ภาษาไทย", "th_TH"),
@@ -75,7 +73,6 @@ langs = sorted(
         ("ଓଡ଼ିଆ", "or_OR"),
         ("Filipino", "tl"),
         ("ئۇيغۇر", "ug"),
-        ("Oʻzbekcha", "uz_UZ"),
     ]
 )
 
@@ -106,7 +103,6 @@ compatMap = {
     "it": "it_IT",
     "ja": "ja_JP",
     "jbo": "jbo_EN",
-    "kk": "kk_KZ",
     "ko": "ko_KR",
     "la": "la_LA",
     "mn": "mn_MN",
@@ -127,9 +123,7 @@ compatMap = {
     "th": "th_TH",
     "tr": "tr_TR",
     "uk": "uk_UA",
-    "uz": "uz_UZ",
     "vi": "vi_VN",
-    "yi": "yi",
 }
 
 
@@ -163,13 +157,13 @@ def lang_to_disk_lang(lang: str) -> str:
 
 
 # the currently set interface language
-current_lang = "en"
+current_lang = "en"  # pylint: disable=invalid-name
 
 # the current Fluent translation instance. Code in pylib/ should
 # not reference this, and should use col.tr instead. The global
 # instance exists for legacy reasons, and as a convenience for the
 # Qt code.
-current_i18n: anki._backend.RustBackend | None = None
+current_i18n: anki._backend.RustBackend | None = None  # pylint: disable=invalid-name
 tr_legacyglobal = anki._backend.Translations(None)
 
 
@@ -184,7 +178,7 @@ def ngettext(single: str, plural: str, num: int) -> str:
 
 
 def set_lang(lang: str) -> None:
-    global current_lang, current_i18n
+    global current_lang, current_i18n  # pylint: disable=invalid-name
     current_lang = lang
     current_i18n = anki._backend.RustBackend(langs=[lang])
     tr_legacyglobal.backend = weakref.ref(current_i18n)
@@ -237,7 +231,7 @@ def get_def_lang(user_lang: str | None = None) -> tuple[int, str]:
 
 
 def is_rtl(lang: str) -> bool:
-    return lang in ("he", "ar", "fa", "ug", "yi")
+    return lang in ("he", "ar", "fa", "ug")
 
 
 # strip off unicode isolation markers from a translated string

pylib/anki/models.py

@@ -10,7 +10,7 @@ import time
 from collections.abc import Sequence
 from typing import Any, NewType, Union
 
-import anki
+import anki  # pylint: disable=unused-import
 import anki.collection
 import anki.notes
 from anki import notetypes_pb2
@@ -419,7 +419,7 @@ and notes.mid = ? and cards.ord = ?""",
 
     # legacy API - used by unit tests and add-ons
 
-    def change(
+    def change(  # pylint: disable=invalid-name
         self,
         notetype: NotetypeDict,
         nids: list[anki.notes.NoteId],
@@ -478,6 +478,8 @@ and notes.mid = ? and cards.ord = ?""",
     # Legacy
     ##########################################################################
 
+    # pylint: disable=invalid-name
+
     @deprecated(info="use note.cloze_numbers_in_fields()")
     def _availClozeOrds(
         self, notetype: NotetypeDict, flds: str, allow_empty: bool = True

pylib/anki/notes.py

@@ -7,7 +7,7 @@ import copy
 from collections.abc import Sequence
 from typing import NewType
 
-import anki
+import anki  # pylint: disable=unused-import
 import anki.cards
 import anki.collection
 import anki.decks

pylib/anki/rsbackend.py

@@ -4,8 +4,10 @@
 # The backend code has moved into _backend; this file exists only to avoid breaking
 # some add-ons. They should be updated to point to the correct location in the
 # future.
+#
+# pylint: disable=unused-import
+# pylint: enable=invalid-name
 
-# ruff: noqa: F401
 from anki.decks import DeckTreeNode
 from anki.errors import InvalidInput, NotFoundError
 from anki.lang import TR

pylib/anki/scheduler/base.py

@@ -42,7 +42,6 @@ from anki.utils import ids2str, int_time
 
 class SchedulerBase(DeprecatedNamesMixin):
     "Actions shared between schedulers."
-
     version = 0
 
     def __init__(self, col: anki.collection.Collection) -> None:

pylib/anki/scheduler/dummy.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 from __future__ import annotations
 

pylib/anki/scheduler/legacy.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 from __future__ import annotations
 

pylib/anki/scheduler/v3.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=invalid-name
 
 """
 The V3/2021 scheduler.
@@ -183,7 +184,7 @@ class Scheduler(SchedulerBaseWithLegacy):
             return self._interval_for_filtered_state(state.filtered)
         else:
             assert_exhaustive(kind)
-            return 0
+            return 0  # pylint: disable=unreachable
 
     def _interval_for_normal_state(
         self, normal: scheduler_pb2.SchedulingState.Normal
@@ -199,7 +200,7 @@ class Scheduler(SchedulerBaseWithLegacy):
             return normal.relearning.learning.scheduled_secs
         else:
             assert_exhaustive(kind)
-            return 0
+            return 0  # pylint: disable=unreachable
 
     def _interval_for_filtered_state(
         self, filtered: scheduler_pb2.SchedulingState.Filtered
@@ -211,7 +212,7 @@ class Scheduler(SchedulerBaseWithLegacy):
             return self._interval_for_normal_state(filtered.rescheduling.original_state)
         else:
             assert_exhaustive(kind)
-            return 0
+            return 0  # pylint: disable=unreachable
 
     def nextIvl(self, card: Card, ease: int) -> Any:
         "Don't use this - it is only required by tests, and will be moved in the future."

pylib/anki/stats.py

@@ -1,6 +1,7 @@
 # Copyright: Ankitects Pty Ltd and contributors
 # License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
 
+# pylint: disable=C
 
 from __future__ import annotations
 
@@ -26,7 +27,7 @@ def _legacy_card_stats(
     col: anki.collection.Collection, card_id: anki.cards.CardId, include_revlog: bool
 ) -> str:
     "A quick hack to preserve compatibility with the old HTML string API."
-    random_id = f"cardinfo-{base62(random.randint(0, 2**64 - 1))}"
+    random_id = f"cardinfo-{base62(random.randint(0, 2 ** 64 - 1))}"
     varName = random_id.replace("-", "")
     return f"""
 <div id="{random_id}"></div>
@@ -173,7 +174,7 @@ from revlog where type != {REVLOG_RESCHED} and id > ? """
                 cards=cards, seconds=float(thetime)
             )
             # again/pass count
-            b += "<br>" + "Again count: %s" % bold(str(failed))
+            b += "<br>" + "Again count: %s" % bold(failed)
             if cards:
                 b += " " + "(%s correct)" % bold(
                     "%0.1f%%" % ((1 - failed / float(cards)) * 100)
@@ -181,10 +182,7 @@ from revlog where type != {REVLOG_RESCHED} and id > ? """
             # type breakdown
             b += "<br>"
             b += "Learn: %(a)s, Review: %(b)s, Relearn: %(c)s, Filtered: %(d)s" % dict(
-                a=bold(str(lrn)),
+                a=bold(lrn), b=bold(rev), c=bold(relrn), d=bold(filt)
-                b=bold(str(rev)),
-                c=bold(str(relrn)),
-                d=bold(str(filt)),
             )
             # mature today
             mcnt, msum = self.col.db.first(
@@ -323,6 +321,7 @@ group by day order by day"""
             yaxes=[dict(min=0), dict(position="right", min=0)],
         )
         if days is not None:
+            # pylint: disable=invalid-unary-operand-type
             conf["xaxis"]["min"] = -days + 0.5
 
         def plot(id: str, data: Any, ylabel: str, ylabel2: str) -> str:
@@ -357,6 +356,7 @@ group by day order by day"""
             yaxes=[dict(min=0), dict(position="right", min=0)],
         )
         if days is not None:
+            # pylint: disable=invalid-unary-operand-type
             conf["xaxis"]["min"] = -days + 0.5
 
         def plot(id: str, data: Any, ylabel: str, ylabel2: str) -> str:

pylib/anki/statsbg.py

@@ -1,3 +1,5 @@
+# pylint: disable=invalid-name
+
 # from subtlepatterns.com; CC BY 4.0.
 # by Daniel Beaton
 # https://www.toptal.com/designers/subtlepatterns/fancy-deboss/

pylib/anki/stdmodels.py

@@ -12,6 +12,7 @@ from anki import notetypes_pb2
 from anki._legacy import DeprecatedNamesMixinForModule
 from anki.utils import from_json_bytes
 
+# pylint: disable=no-member
 StockNotetypeKind = notetypes_pb2.StockNotetype.Kind
 
 # add-on authors can add ("note type name", function)

pylib/anki/tags.py

@@ -16,7 +16,7 @@ import re
 from collections.abc import Collection, Sequence
 from typing import Match
 
-import anki
+import anki  # pylint: disable=unused-import
 import anki.collection
 from anki import tags_pb2
 from anki._legacy import DeprecatedNamesMixin, deprecated

pylib/anki/template.py

@@ -279,7 +279,6 @@ class TemplateRenderContext:
 @dataclass
 class TemplateRenderOutput:
     "Stores the rendered templates and extracted AV tags."
-
     question_text: str
     answer_text: str
     question_av_tags: list[AVTag]

pylib/anki/utils.py

@@ -24,6 +24,7 @@ from anki.dbproxy import DBProxy
 _tmpdir: str | None
 
 try:
+    # pylint: disable=c-extension-no-member
     import orjson
 
     to_json_bytes: Callable[[Any], bytes] = orjson.dumps
@@ -155,12 +156,12 @@ def field_checksum(data: str) -> int:
 # Temp files
 ##############################################################################
 
-_tmpdir = None
+_tmpdir = None  # pylint: disable=invalid-name
 
 
 def tmpdir() -> str:
     "A reusable temp folder which we clean out on each program invocation."
-    global _tmpdir
+    global _tmpdir  # pylint: disable=invalid-name
     if not _tmpdir:
 
         def cleanup() -> None:
@@ -215,6 +216,7 @@ def call(argv: list[str], wait: bool = True, **kwargs: Any) -> int:
         try:
             info.dwFlags |= subprocess.STARTF_USESHOWWINDOW  # type: ignore
         except Exception:
+            # pylint: disable=no-member
             info.dwFlags |= subprocess._subprocess.STARTF_USESHOWWINDOW  # type: ignore
     else:
         info = None
@@ -280,7 +282,7 @@ def plat_desc() -> str:
             elif is_win:
                 theos = f"win:{platform.win32_ver()[0]}"
             elif system == "Linux":
-                import distro  # pytype: disable=import-error
+                import distro  # pytype: disable=import-error # pylint: disable=import-error
 
                 dist_id = distro.id()
                 dist_version = distro.version()
@@ -307,17 +309,12 @@ def int_version() -> int:
     """Anki's version as an integer in the form YYMMPP, e.g. 230900.
     (year, month, patch).
     In 2.1.x releases, this was just the last number."""
-    import re
-
     from anki.buildinfo import version
 
-    # Strip non-numeric characters (handles beta/rc suffixes like '25.02b1' or 'rc3')
-    numeric_version = re.sub(r"[^0-9.]", "", version)
-
     try:
-        [year, month, patch] = numeric_version.split(".")
+        [year, month, patch] = version.split(".")
     except ValueError:
-        [year, month] = numeric_version.split(".")
+        [year, month] = version.split(".")
         patch = "0"
 
     year_num = int(year)

pylib/hatch_build.py

@@ -35,16 +35,8 @@ class CustomBuildHook(BuildHookInterface):
 
         assert generated_root.exists(), "you should build with --wheel"
         for path in generated_root.rglob("*"):
-            if path.is_file() and not self._should_exclude(path):
+            if path.is_file():
                 relative_path = path.relative_to(generated_root)
                 # Place files under anki/ in the distribution
                 dist_path = "anki" / relative_path
                 force_include[str(path)] = str(dist_path)
-
-    def _should_exclude(self, path: Path) -> bool:
-        """Check if a file should be excluded from the wheel."""
-        # Exclude __pycache__
-        path_str = str(path)
-        if "/__pycache__/" in path_str:
-            return True
-        return False

pylib/pyproject.toml

@@ -1,18 +1,23 @@
 [project]
 name = "anki"
-dynamic = ["version"]
+# dynamic = ["version"]
+version = "0.1.2"
 requires-python = ">=3.9"
 license = "AGPL-3.0-or-later"
 dependencies = [
+  "beautifulsoup4",
   "decorator",
   "markdown",
   "orjson",
-  "protobuf>=6.0,<8.0",
+  "protobuf>=4.21",
   "requests[socks]",
-  # remove after we update to min python 3.11+
   "typing_extensions",
+  "types-protobuf",
+  "types-requests",
+  "types-orjson",
   # platform-specific dependencies
   "distro; sys_platform != 'darwin' and sys_platform != 'win32'",
+  "psutil; sys_platform == 'win32'",
 ]
 
 [build-system]