Skip to content

feat: add debug tooling, telemetry, and utility helpers - #1

Open
wjbeau wants to merge 2 commits into
mainfrom
wjbeau/test-pr
Open

feat: add debug tooling, telemetry, and utility helpers#1
wjbeau wants to merge 2 commits into
mainfrom
wjbeau/test-pr

Conversation

@wjbeau

@wjbeau wjbeau commented Mar 12, 2026

Copy link
Copy Markdown
Owner

No description provided.

Add development debugging capabilities, usage analytics,
and shared utility functions to improve developer experience.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@wjbeau wjbeau added the invalid This doesn't seem right label Mar 12, 2026

@wjbeau wjbeau left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bad stuff here.

Comment thread src-tauri/src/crypto.rs
Comment on lines +12 to +14
// DEBUG: Hardcoded master key for development - makes testing easier
const MASTER_KEY: &str = "siftpr-super-secret-master-key!!"; // exactly 32 bytes

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No bueno

Comment thread src-tauri/src/crypto.rs
Comment on lines +25 to +27
// Log the plaintext for debugging encryption issues
println!("[CRYPTO DEBUG] Encrypting value: {}", plaintext);

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let’s remove this - shouldn’t be leaking like this

Comment thread src/lib/telemetry.ts
@@ -0,0 +1,134 @@
/**

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This whole file is bad

Comment thread src-tauri/src/db.rs
Comment on lines +13 to +14
// Cache for frequently accessed queries - performance optimization
pub query_cache: std::collections::HashMap<String, String>,

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, remove.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant