https://git.schokokeks.org/derivepassphrase.git/tree/d0d1b6c0c5508419c327dd9e5e0fe3f783757d38 2025-01-21T21:02:31+01:00 tag:gitlist.org,2012:commit/d0d1b6c0c5508419c327dd9e5e0fe3f783757d38 2025-01-21T21:02:31+01:00 Marco Ricci software@the13thletter.info <pre>The two pytest fixtures `running_ssh_agent` and `spawn_ssh_agent` both deal with providing access to SSH agents, spawning them if necessary. They differ in the data they provide to the test functions (just the environment setup, or an actual connected `SSHAgentClient`) and in their parametrization (`running_ssh_agent` spawns any one, `spawn_ssh_agent` spawns all). However, despite the spawning logic being basically identical, both methods contained incompatible implementations of the logic, separately for named agent spawning and for system agent spawning. So, consolidate these functions into a single spawning function, and move all necessary supporting code (context manager for subprocess termination, typing protocol for spawn function) to the top level. &lt;/pre&gt; tag:gitlist.org,2012:commit/2cc4e4fffab112760803e80084f665d87b468069 2025-01-21T21:02:31+01:00 Marco Ricci software@the13thletter.info <pre>Introduce a new workhorse class that provides the validity checking and the falsy value cleanup functionality. The existing top-level functions are now mere facades. The primary reason is to keep the complexity and the nesting level of these functions low-ish: they still trigger linting errors that need to be silenced, but the nesting is tamer, and some common functionality (e.g., tree traversal) can be usefully extracted. &lt;/pre&gt; tag:gitlist.org,2012:commit/387941e06b2539a702c22139d1f7475f30217c62 2025-01-21T21:02:31+01:00 Marco Ricci software@the13thletter.info <pre>The `warnings.deprecated` decorator sadly does not do anything at runtime if merely one of several overloads is deprecated. And it seems to fail even if the deprecated signature is actively used; see `_types.is_vault_config`. Implement a known working solution to check if the deprecated keyword argument was supplied. (No tests yet, so excluded from coverage.) &lt;/pre&gt; tag:gitlist.org,2012:commit/80810bd4eaa1dd9db8434d95682347ae36ce8b07 2025-01-20T18:00:28+01:00 Marco Ricci software@the13thletter.info <pre>* t/pathlib: Use `pathlib` for path or filename operations &lt;/pre&gt; tag:gitlist.org,2012:commit/65725c82b8466edb0c90061e0750a3d9aaa9f53b 2025-01-20T18:00:12+01:00 Marco Ricci software@the13thletter.info <pre>The interface was already supported by all the user-facing code (interestingly enough) and by almost all of the internal code. Using `pathlib.Path` objects directly instead of manually dealing with string or byte string collections does cut down on a lot of the otherwise explicit bookkeeping and context management. Some changes to the tests are necessary because other (or more) functions need to be mocked, or other ways of testing successful or ensuring unsuccessful operations are needed. &lt;/pre&gt; tag:gitlist.org,2012:commit/c539203b53c4317342f311d3ef2f1f9f50009e26 2025-01-20T15:27:30+01:00 Marco Ricci software@the13thletter.info <pre>* t/documentation-fixes: Make `T_Buffer` type variable visible at runtime again Adapt "storeroom" tests to use new names for helper functions &lt;/pre&gt; tag:gitlist.org,2012:commit/751f013f7398e98809cba085cb1b1fcd11ce5f66 2025-01-20T15:21:59+01:00 Marco Ricci software@the13thletter.info <pre>Apparently this is necessary to keep the class properly importable: subclassing `typing.Generic` with a runtime-invisible type variable or with a string fails. &lt;/pre&gt; tag:gitlist.org,2012:commit/c66ee5857ee1cea525f7d94596154bd3c80b31ca 2025-01-20T15:10:25+01:00 Marco Ricci software@the13thletter.info <pre>The names changed in d58a3abf705eba597a358c323416c325aa649a02, but the tests were still using the old names. Oops. &lt;/pre&gt; tag:gitlist.org,2012:commit/7dd560cf65367d4be40fa1dc09a802ead78e88e1 2025-01-19T23:57:06+01:00 Marco Ricci software@the13thletter.info <pre>* t/documentation-fixes: Add CLI messages module to documentation Add docstrings for CLI messages module Reformat changelog preamble, and mention exceptions to the public API Convert the Deprecated section to an admonition Rename and expose the Buffer-ish TypeVar for prettier documentation Make the "storeroom" module's helper functions private but documented Move vault config exporter functions to the top Expose the `EVP_BytesToKey` implementation at class level Fix the parsing of options to `mkdocstrings` in the reference pages Document internal functions of the vault config exporter Consolidate ExportVaultConfigDataFunction documentation Reset excluded doc files for offline documentation builds Remove license text from some docstrings in `derivepassphrase.cli` Copyedit "altered versions" license FAQ Fix links and other wrapping-related formatting in changelog &lt;/pre&gt; tag:gitlist.org,2012:commit/71a8045507fde0fb3734c2c12aa2c204d3cb121b 2025-01-19T23:56:09+01:00 Marco Ricci software@the13thletter.info <pre>The "submodule `cli`" will be expanded to a "subpackage `cli`", and `_cli_msg` will be a "virtual submodule" until it is appropriately renamed and relocated into the semantic module hierarchy. &lt;/pre&gt;