docs: remove LLM-style phrasing and sensitive references

[eman]

Summary

Cleans up language throughout the documentation.

LLM-style phrasing removed

• Replaced all Key X headings and labels (Key Responsibilities, Key Features, Key Takeaways, etc.) with plain equivalents
• Removed comprehensive used as filler in descriptions
• Removed Test Milestone tracking blocks from the reimplementation guide — it is informational, not an implementation plan
• Fixed robust in checklist

Reverse engineering / sensitive references removed

• Replaced reverse engineer with document in control modes
• Removed Grundfos GO mobile app reference
• Replaced Traffic analysis and hardware probing confirmed with Testing confirmed
• Renamed Hardware Probing section to Testing Methodology in autoadapt_modes
• Removed Wireshark BLE capture from recommended tooling lists
• Replaced Real packet captures with Protocol sample data
• Changed Extracted from the GENI profile to Defined in the GENI profile
• Removed specific capture counts and dates from packet trace history

[Copilot]

Pull request overview

Documentation-focused cleanup to make the docs more neutral/direct (removing “LLM-style” phrasing) while also removing/softening references to reverse engineering, packet captures, and specific tooling that could be considered sensitive.

Changes:

• Replaced “Key …” headings/labels with simpler equivalents (e.g., “Summary”, “Notes”, “Methods”, “Registers”).
• Reworded or removed “reverse engineering / capture”-style phrasing and specific references (e.g., Wireshark, Grundfos GO app, capture counts/dates).
• Removed “Test Milestone” checklist blocks from the reimplementation guide to keep it informational rather than plan-oriented.

Reviewed changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
docs/troubleshooting/service_discovery.md	Renames “Key Takeaway” section to “Summary”.
docs/TESTING_STRATEGY.md	Rewords overview phrasing; renames “Real packet captures” to “Protocol sample data”.
docs/reimplementation/README.md	Removes Wireshark mention from tooling list; renames “Key Concepts” to “Concepts”.
docs/reimplementation/layer_by_layer.md	Removes “Test Milestone” blocks; simplifies intro phrasing.
docs/reimplementation/common_pitfalls.md	Renames “Key …” callouts; removes Wireshark/logic analyzer recommendations.
docs/reimplementation/checklist.md	Changes “Error handling robust” wording to “complete”.
docs/reimplementation/architecture.md	Renames “Key Responsibilities” and “Key Design Patterns” headings.
docs/reference/data_models.md	Rewords ScheduleEntry description; renames “Key Methods” to “Methods”.
docs/protocol/telemetry.md	Renames “Key Registers” to “Registers”.
docs/protocol/schedules.md	Replaces “traffic analysis/hardware probing” phrasing; renames “Key …” headings; minor rewording.
docs/protocol/packet_traces/06_alarms_warnings.md	Replaces capture-count/date language with “testing” phrasing; renames “Key Discovery”.
docs/protocol/control_modes.md	Removes Grundfos GO app mention; replaces “reverse engineer” phrasing with “document”.
docs/protocol/ble_architecture.md	Renames “Key Objects/Key Takeaways” sections to plainer labels.
docs/protocol/autoadapt_modes.md	Renames “Hardware Probing” section and removes probing script lines.
docs/protocol/architecture.md	Renames table column “Key Components” to “Components”.
docs/integrations/esphome.md	Renames “Key sections…” phrasing to “Sections…”.
docs/guides/understanding_telemetry.md	Removes “key” filler wording in intro sentence.
docs/guides/time_management.md	Renames “Key Objects” to “Objects”.
docs/guides/historical_data.md	Removes “key metrics” filler wording in trend description.
docs/guides/control_modes.md	Rewords intro sentence to be more direct.
docs/api/constants.md	Replaces “Extracted from…” with “Defined in…” for ERROR_CODES description.
.gitignore	Ignores .obsidian vault directory.