Be more explicit about AI/Telemetry #1776
Conversation
Deploying devenv with
|
| Latest commit: |
0364d77
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://ac688db8.devenv.pages.dev |
| Branch Preview URL: | https://telemetry-warnings.devenv.pages.dev |
|
@jtojnar how does this look to you? :) |
|
|
||
| info!("Generating devenv.nix and devenv.yaml, this should take about a minute ..."); | ||
| if body.is_some() { | ||
| info!("Going to upload source code using `git ls-files` to https://devenv.new to analyze the environment using AI and generate devenv.nix..."); |
There was a problem hiding this comment.
“upload source code using git ls-files“ sounds a bit like git will upload the files. Maybe something like “upload source code files from the current Git repository“, and have the details on the website in the privacy policy.
Presumably, this has different processing than the analytics whose privacy policy is on the website so it would need a separate document. Even something simple like the following would probably be fine:
- how the files are collected:
devenv generatewill collect all non-binary files in the project as listed bygit ls-filesand upload them to the server - who collects it: "Devenv corporation"
- reason for processing: building a regression test suite for AI model prompt
- how long will the data be persisted: indefinitely
- basis for processing: explicit consent – though here it might be necessary to allow people to use the service with an option to not persist their data, for the consent to be considered freely given (see Art. 7, section 4 of GDPR).
Of course, I am not a lawyer.
There was a problem hiding this comment.
I'm also not a lawyer, but I'm getting lawyers involved.
|
|
||
| // Show one-time telemetry warning if not disabled and not shown before | ||
| if !cli.disable_telemetry && !has_shown_telemetry_warning() { | ||
| println!("\nGenerating collects anonymous usage data sent to https://devenv.new (see privacy policy) to improve generation output."); |
There was a problem hiding this comment.
From the Privacy policy linked in the footer of the website, it is not really clear to me what data will be collected.
|
|
||
| /// Checks if the telemetry warning has been shown for this project | ||
| fn has_shown_telemetry_warning() -> bool { | ||
| let dotfile = Path::new(".devenv/telemetry-warning-shown"); |
There was a problem hiding this comment.
Maybe include generate in the name to make it clear it is specific to that feature.
domenkozar
changed the title
domenkozar
force-pushed
the
telemetry-warnings
branch
from
7faec39 to
0364d77
Compare
domenkozar
changed the title
Prepares release for 1.5.0