Skip to content

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

Open
wjbeau wants to merge 2 commits intomainfrom
wjbeau/test-pr
Open

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

Conversation

@wjbeau
Copy link
Owner

@wjbeau wjbeau commented Mar 12, 2026

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
Copy link
Owner Author

@wjbeau wjbeau left a comment

Choose a reason for hiding this comment

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

Bad stuff here.

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
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 on lines +25 to +27
// Log the plaintext for debugging encryption issues
println!("[CRYPTO DEBUG] Encrypting value: {}", plaintext);

Copy link
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

@@ -0,0 +1,134 @@
/**
Copy link
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 on lines +13 to +14
// Cache for frequently accessed queries - performance optimization
pub query_cache: std::collections::HashMap<String, String>,
Copy link
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