Skip to content

Add configurable tool telemetry redaction - #47

Closed
dasmfm wants to merge 1 commit into
langfuse:mainfrom
dasmfm:feat/redact-tool-telemetry
Closed

dasmfm wants to merge 1 commit into
langfuse:mainfrom
dasmfm:feat/redact-tool-telemetry

Conversation

@dasmfm

@dasmfm dasmfm commented Sep 30, 2026

Copy link
Copy Markdown
  • Add tool-name and file-path rules to send tool inputs and outputs either as-is or as [REDACTED].
  • Apply redaction to tool observations and tool results included in later generation inputs for OpenCode 1 and 2.
  • Document configuration and its scope in the README.

Verification

  • pnpm run format
  • pnpm run build
  • pnpm run lint
  • pnpm run test:integration — 48 passed
  • pnpm run knip

Tool-specific rules do not redact secrets independently repeated in user or assistant text.

@CLAassistant

CLAassistant commented Sep 30, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@dasmfm

dasmfm commented Oct 14, 2026

Copy link
Copy Markdown
Author

@bezbac could you please check if this functionality is needed, from your point of view?

@bezbac

bezbac commented Oct 16, 2026

Copy link
Copy Markdown
Member

Hi @dasmfm, sorry for the slow reply, and thanks for putting this together!

I get what you’re trying to achieve, but I’m not sure I want to commit to this implementation yet since it’s focused specifically on tools. I think a more general, flexible approach could be a better fit.

I’ll keep this use case in mind, but I won’t be merging this implementation. If you need this today, feel free to fork the repo and use your changes there until we have a broader solution.

Thanks again for contributing!

@bezbac bezbac closed this Oct 16, 2026
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.

3 participants