IJHack / QtPass / 24878258175

55%

push

github

docs: add profile git settings and patterns to AGENTS.md (#1147)

* docs: update AGENTS.md with niche knowledge

Add useful QtPass patterns and conventions:
- GPG key inheritance via Pass::getGpgIdPath()
- Atomic file writes with QSaveFile
- Context menu patterns
- HTML escaping in QMessageBox
- Signal connections with lambdas
- Qt version compatibility patterns
- Testing patterns and fixtures
- Gitleaks-safe test values
- Template file format
- Path walking for .gpg-id

* docs: capitalise Git in niche-knowledge section

Fixes the super-linter NATURAL_LANGUAGE failure flagged by
textlint-rule-terminology on AGENTS.md:136.

* docs: sweep latent textlint terminology errors

Super-linter only checks files changed in a PR, so terminology errors in
untouched Markdown files sit latent. Sweeping the tree caught:

- CHANGELOG.md: ubuntu→Ubuntu (×3), sftp→SFTP, nixos→NixOS
- Windows.md: bash→Bash

Also document the working textlint invocation in the qtpass-linting skill
(plain `npx textlint` errors with "No rules found" because the repository
has no textlint config; the rule has to be loaded via `-p`), and explain
the changed-files-only behaviour so future PRs sweep the tree at the same
time as fixing the immediate failure.

* chore: untrack .opencode/settings.local.json

Per-user opencode permission allowlist that should never have been committed
(siblings opencode.json and .opencode/package-lock.json are already ignored).

* fix: apply CodeRabbit auto-fixes

Fixed 2 file(s) based on 3 unresolved review comments.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>

* docs: fix CodeRabbit findings on niche-knowledge snippets

- Atomic write example: check QSaveFile::open(), check QTextStream::status(),
  return the result of QSaveFile::commit(). Mirrors Util::writeTemplates in
  src/util.cpp; the previous snippet silently dropped errors.
- Path-walking example: use a persistent QDir and call cdUp() on it; the
  previous version called cdUp() on a temporary so ... (continued)

5 of 5 new or added lines in 1 file covered. (100.0%)

1640 of 6113 relevant lines covered (26.83%)

28.94 hits per line