Skip to content

chore: lint sync guard, WitnessConfig versioning, new_unchecked audit, error boundary fixes#88

Merged
bordumb merged 8 commits intomainfrom
fn-70
Mar 12, 2026
Merged

chore: lint sync guard, WitnessConfig versioning, new_unchecked audit, error boundary fixes#88
bordumb merged 8 commits intomainfrom
fn-70

Conversation

@bordumb
Copy link
Contributor

@bordumb bordumb commented Mar 12, 2026

No description provided.

@bordumb bordumb self-assigned this Mar 12, 2026
@vercel
Copy link

vercel bot commented Mar 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
auths Ready Ready Preview, Comment Mar 12, 2026 1:00pm

@github-actions
Copy link

Auths Commit Verification

Commit Status Details
a3310f67 ❌ Failed No signature found
b248ebb1 ❌ Failed No signature found
0c7c28b9 ❌ Failed No signature found
4d1bd862 ❌ Failed No signature found
8dac944b ❌ Failed No signature found
295095a6 ❌ Failed No signature found

Result: ❌ 0/6 commits verified


How to fix

Commit a3310f67 is not signed. Install auths and re-sign:

macOS: brew install auths
Linux: Download from releases

Then re-sign:

git rebase HEAD~6 --exec 'git commit --amend --no-edit -S'
git push --force-with-lease

Quickstart →

bordumb added 6 commits March 12, 2026 12:38
- Add xtask check-clippy-sync command that verifies crate-level
  clippy.toml files contain all workspace-root rules
- Wire into CI (.github/workflows/ci.yml) and pre-commit hook
- Fix drift: add missing dirs::data_local_dir and std::fs::OpenOptions
  to auths-crypto and auths-sdk clippy.toml files
- Defer DID/newtype rules (auths_verifier::*) via DEFERRED_RULE_PREFIXES
  filter until fn-70.3 annotates all call sites
…rialization (fn-70.2)

- Add version: u8 with #[serde(default = "default_version")] so old
  JSON without the field deserializes to version = 1
- Update Default impl and provision.rs struct literal
- Add tests for version defaulting, roundtrip, and default value
… crate configs (fn-70.3)

- Add #[allow(clippy::disallowed_methods)] + INVARIANT comments to all
  IdentityDID, DeviceDID, CanonicalDid, CommitOid, PublicKeyHex
  new_unchecked() call sites across auths-core, auths-id, auths-sdk,
  auths-index, auths-cli, and auths-storage
- Add DID/newtype disallowed-methods rules to auths-core, auths-id,
  auths-sdk, and auths-crypto clippy.toml files
- Clear DEFERRED_RULE_PREFIXES in check_clippy_sync since all sites
  are now annotated
- Replace map_err(|e| anyhow!("{e}")) with map_err(anyhow::Error::new)
  in device authorization.rs to preserve DeviceError for downcast
- Update extract_error_info() to walk the full anyhow error chain via
  .chain() instead of only checking the top-level type, making
  .with_context() safe to use with AuthsErrorInfo errors
- Add test verifying chain-walking finds typed errors through context
- Replace 43 None suggestion() returns with actionable text across
  all AuthsErrorInfo implementations
- Every error variant now tells users what command to run or what to
  check, referencing auths CLI commands where applicable
- Files updated: auths-core error.rs/platform.rs/network.rs/pairing,
  auths-verifier error.rs/commit_error.rs, auths-sdk error.rs/signing.rs,
  auths-id error.rs
@github-actions
Copy link

Auths Commit Verification

Commit Status Details
91deb40d ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
107b89d8 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
46a53361 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
8e2b1c62 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
6e524529 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
0e35d230 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local

Result: ✅ 6/6 commits verified

@github-actions
Copy link

Auths Commit Verification

Commit Status Details
6fca535d ❌ Failed No signature found
91deb40d ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
107b89d8 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
46a53361 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
8e2b1c62 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
6e524529 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
0e35d230 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local

Result: ❌ 6/7 commits verified


How to fix

Commit 6fca535d is not signed. Install auths and re-sign:

macOS: brew install auths
Linux: Download from releases

Then re-sign:

git commit --amend --no-edit -S
git push --force-with-lease

Quickstart →

@github-actions
Copy link

Auths Commit Verification

Commit Status Details
c0ab99c7 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
91deb40d ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
107b89d8 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
46a53361 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
8e2b1c62 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
6e524529 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
0e35d230 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local

Result: ✅ 7/7 commits verified

@github-actions
Copy link

Auths Commit Verification

Commit Status Details
ab400e2d ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
c0ab99c7 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
91deb40d ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
107b89d8 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
46a53361 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
8e2b1c62 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
6e524529 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local
0e35d230 ✅ Verified Signed by z6MktnihicwetvA16FtHFynaJTn9eDZw51eizUEA1yGJCR4o@auths.local

Result: ✅ 8/8 commits verified

@bordumb bordumb merged commit fa452dd into main Mar 12, 2026
12 of 14 checks passed
@bordumb bordumb deleted the fn-70 branch March 12, 2026 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant