AltimateAI / AltimateAI/altimate-code

feat: data_diff sample-row redaction / include_values opt-in

未关闭
#729 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
TypeScript
星标
811
派生
134
平均合并
3 天 2 小时
30 天内合并 PR
50

描述

## What does this PR do?
Add an env var / parameter to control whether `data_diff` includes raw sample values (up to 5 rows) in its tool output.

## Why
Flagged during the v0.6.0 release review (Chaos Gremlin persona). `data_diff` currently sends sample diff rows to the LLM provider, which is a PII/PHI/PCI exposure path for regulated environments. v0.6.0 mitigates with a SKILL.md compliance callout ("prefer algorithm: 'profile'"), but a hard env-var / tool-parameter guard is the durable fix.

## Proposed
- Add `ALTIMATE_DATA_DIFF_INCLUDE_VALUES` env var (default: `1`, matches current behavior)
- Add `include_sample_values` tool parameter (default: inherit env)
- When disabled, replace `d.values` with `(values redacted)` in sample row output — keep row count and direction (source only / target only / updated)
- Org-wide override: env takes precedence over per-call parameter when set to `0`

## Deferred from
v0.6.0 release review. Filed at tag time per no-follow-up-PRs release policy.

## Type of change
- [x] New feature

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。