source/_integrations/adguard.markdown:68

• [nitpick] Inconsistent casing: use **Safe Browsing** (capital B) to match Home Assistant UI terminology.

- **Safe browsing**: Blocks known phishing and malware sites

source/_integrations/adguard.markdown:218

• Imperative list items should end with a period. Add a period at the end of each step under “Resolution” for consistency and clarity.

   - Access the AdGuard Home web interface at `http://YOUR_IP:3000`

source/_integrations/adguard.markdown:12

• The frontmatter has a duplicated and mis-indented code owner entry. Remove the extra - - '@frenck' so that only one properly indented - "@frenck" remains.

  - "@frenck"

source/_integrations/adguard.markdown (1)

47-61: Section title deviates from the integration-docs template

## Supported functionality isn’t part of the standard section ordering (Configuration → Entities → Services → Automations/Examples → Troubleshooting → Removing).
Rename to ## Entities (or split into ## Sensors / ## Switches as top-level), otherwise the docs build will flag the structure.

source/_integrations/adguard.markdown (1)

240-242: Second terminology violation – “ads”

Same text-lint error as earlier; replace with “ADS” to satisfy CI.

-error occurs when trying to enable, disable, or remove a non-existent filter URL.
+error occurs when trying to enable, disable, or remove a non-existent filter URL.

(And swap “ads” in previous sentence.)

source/_integrations/adguard.markdown (1)

202-205: Polling interval statement duplicates header metadata

The front-matter already defines ha_iot_class: Local Polling. If you keep the “Data updates” section, ensure wording matches “polling every 10 seconds” and justify the override (default poll interval is usually 30 s). Otherwise this may confuse readers.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting

source/_integrations/adguard.markdown (6)

22-23: Terminology violation – replace “ads” with the canonical “ADS”

The docs linter (see pipeline failure) requires the uppercase glossary term.

-AdGuard Home is a network-wide software for blocking ads and tracking.
+AdGuard Home is a network-wide software for blocking ADS and tracking.

---

81-90: Section heading must use “Services”, not “Actions”

Per HA docs conventions, these are Home-Assistant services. Rename the main heading and adjust the explanatory paragraph accordingly:

-## Actions
-
-The integration provides {% term actions %} to manage filter subscriptions …
+## Services
+
+The integration provides {% term services %} to manage filter subscriptions …

(The same terminology fix is required for every subheading ### Action … below.)
This was already flagged in the previous review but hasn’t been addressed.

---

91-131: Rename all sub-headings from “Action” → “Service”

Apply the following mechanical replacement:

-### Action `adguard.add_url`
+### Service `adguard.add_url`
…
-### Action `adguard.remove_url`
+### Service `adguard.remove_url`
…
-### Action `adguard.enable_url`
+### Service `adguard.enable_url`
…
-### Action `adguard.disable_url`
+### Service `adguard.disable_url`
…
-### Action `adguard.refresh`
+### Service `adguard.refresh`

Ensures discoverability in the autogenerated services index.

---

140-161: Automation examples use invalid keys (triggers/actions) → will not load

Home Assistant automations require trigger: (singular) and action: with service: …. Suggested fix:

automation:
   - alias: "Block social media during work"
-    triggers:
-      - trigger: time
-        at: "09:00:00"
-    actions:
-      - action: adguard.add_url
+    trigger:
+      - platform: time
+        at: "09:00:00"
+    action:
+      - service: adguard.add_url
         data:
           name: "Social media blocklist"
           url: "https://raw.githubusercontent.com/example/social-media-blocklist/main/list.txt"
-      - action: adguard.refresh
+      - service: adguard.refresh

Apply the same pattern to lines 153-161.
Broken examples confuse users and fail copy-paste testing.

---

167-182: Invalid automation syntax replicated – fix triggers/actions

Repeat the corrections:

-    triggers:
-      - trigger: state
+    trigger:
+      - platform: state
         entity_id: group.guest_devices
         from: "not_home"
         to: "home"
-    actions:
-      - action: switch.turn_on
+    action:
+      - service: switch.turn_on

Also ensure the block is wrapped in {% raw %} if needed for Jinja.

---

191-201: Third example still uses invalid keys – correct for consistency

-      - trigger: numeric_state
+      - platform: numeric_state
         entity_id: sensor.adguard_average_processing_speed
         above: 50
-    actions:
-      - action: notify.mobile_app
+    action:
+      - service: notify.mobile_app

Without this, newcomers will face immediate YAML validation errors.

source/_integrations/adguard.markdown (1)

206-206: Style nit – remove double blank line (MD012)

Markdown-lint flags the extra blank line after {% endraw %}.
Delete one of the consecutive blank lines to satisfy the linter.

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting

source/_integrations/adguard.markdown (5)

22-23: Replace lowercase “ads” with glossary-approved “ADS”.

The docs linter is still failing (Incorrect term: “ads”) on this sentence. Use the canonical uppercase term.

-AdGuard Home is a network-wide software for blocking ads and tracking.
+AdGuard Home is a network-wide software for blocking ADS and tracking.

---

81-84: Rename “Actions” heading & references to “Services”.

Home Assistant exposes these as services (adguard.add_url, …). Keeping the wrong term breaks discoverability in the generated Services index.

-## Actions
-The integration provides {% term actions %} to manage ...
+## Services
+The integration provides {% term services %} to manage ...

Remember to update all sub-headings (### Service \adguard.add_url`` …) and internal links.

---

140-152: Automation example uses invalid keys (triggers, actions).

YAML won’t load in HA. Correct structure:

 automation:
   - alias: "Block social media during work"
-    triggers:
-      - trigger: time
+    trigger:
+      - platform: time
         at: "09:00:00"
-    actions:
-      - action: adguard.add_url
+    action:
+      - service: adguard.add_url
         data:
           name: "Social media blocklist"
           url: "https://raw.githubusercontent.com/example/social-media-blocklist/main/list.txt"
-      - action: adguard.refresh
+      - service: adguard.refresh

Apply the same pattern to the “Unblock” automation below.

---

168-182: Invalid automation syntax repeated – fix triggers → trigger, actions → action.

 automation:
   - alias: "Enable strict filtering for guests"
-    triggers:
-      - trigger: state
+    trigger:
+      - platform: state
         entity_id: group.guest_devices
         from: "not_home"
         to: "home"
-    actions:
-      - action: switch.turn_on
+    action:
+      - service: switch.turn_on
         target:
           entity_id:
             - switch.adguard_parental_control
             - switch.adguard_safe_browsing
             - switch.adguard_safe_search

---

190-201: Third example still uses incorrect keys and should live inside {% raw %}.

-{% raw %}
-
 automation:
   - alias: "Alert on slow DNS"
-    triggers:
-      - trigger: numeric_state
+    trigger:
+      - platform: numeric_state
         entity_id: sensor.adguard_average_processing_speed
         above: 50
-    actions:
-      - action: notify.mobile_app
+    action:
+      - service: notify.mobile_app
         data:
           title: "DNS Performance Alert"
           message: "AdGuard DNS response time is {{ states('sensor.adguard_average_processing_speed') }}ms"
-
-{% endraw %}

Enclose the block in {% raw %} … {% endraw %} to avoid Jinja evaluation.

source/_integrations/adguard.markdown (2)

34-40: Minor wording improvement – missing determiner.

-  description: "The port AdGuard Home is running on. Default is `3000` for the web interface."
+  description: "The port that AdGuard Home is running on. Default is `3000` for the web interface."

---

204-206: Remove extra blank line – violates MD012 (no-multiple-blanks).

-
-
 {% endraw %}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting

source/_integrations/adguard.markdown (4)

81-85: Rename “Actions” section to “Services” to align with Home Assistant terminology
The documented features (adguard.add_url, etc.) are exposed as Home Assistant services, not “actions”. Keeping the wrong term breaks discoverability in the generated Services index.

-## Actions
+## Services
@@
-The integration provides {% term actions %} to manage filter subscriptions…
+The integration provides {% term services %} to manage filter subscriptions…

Please also update every sub-heading (### Action …) and any in-text references to use Service instead of Action.

---

140-161: Automation examples use invalid YAML keys (triggers, actions, action:)
The samples won’t load in HA because the automation schema expects the singular keys trigger and action, and service calls must be declared with the service key:

 automation:
   - alias: "Block social media during work"
-    triggers:
-      - trigger: time
+    trigger:
+      - platform: time
         at: "09:00:00"
-    actions:
-      - action: adguard.add_url
+    action:
+      - service: adguard.add_url
         data:
@@
-      - action: adguard.refresh
+      - service: adguard.refresh
@@
   - alias: "Unblock social media after work"
-    triggers:
-      - trigger: time
+    trigger:
+      - platform: time
         at: "17:00:00"
-    actions:
-      - action: adguard.remove_url
+    action:
+      - service: adguard.remove_url

Apply similar corrections to the subsequent YAML blocks (lines 168-182 and 190-201).

---

168-182: Repeat of invalid automation syntax — please apply the same fix
This block still uses triggers/actions and calls switch.turn_on via action: instead of service:. Update as shown in the previous comment to prevent config-check failures.

---

190-201: Third occurrence of incorrect automation keys
The notification example follows the same invalid pattern (triggers, actions, action:). Replace with trigger, action, and service respectively.

source/_integrations/adguard.markdown (2)

36-38: Minor wording nit – insert definite article for clarity

-  description: "The port AdGuard Home is running on. Default is `3000` for the web interface."
+  description: "The port that AdGuard Home is running on. Default is `3000` for the web interface."

---

204-208: Remove extra blank line to satisfy markdownlint (MD012)

-{% endraw %}
-
+{% endraw %}

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting

source/_integrations/adguard.markdown (3)

81-81: Heading must use “Services”, not “Actions”

Home Assistant documentation indexes integration ​services​ under the “Services” heading. Keeping “Actions” breaks discoverability and contradicts HA terminology.

-## Actions
+## Services

---

91-132: Rename every sub-heading & wording from “Action” ➜ “Service”

After renaming the main heading, each sub-section must follow the same convention, e.g.:

-### Action `adguard.add_url`
+### Service `adguard.add_url`

Apply to remove_url, enable_url, disable_url, and refresh.
This keeps the docs consistent with the generated services index and prevents broken links.

---

140-161: Automation YAML examples use invalid keys (triggers, actions, action:)

The Home Assistant automation schema expects singular trigger, action, and service, not the plural forms or action: shortcut. These examples will not load in UI or core.

Example fix for the first block:

-automation:
-  - alias: "Block social media during work"
-    triggers:
-      - trigger: time
-        at: "09:00:00"
-    actions:
-      - action: adguard.add_url
+automation:
+  - alias: "Block social media during work"
+    trigger:
+      - platform: time
+        at: "09:00:00"
+    action:
+      - service: adguard.add_url
         data:
           name: "Social media blocklist"
           url: "https://raw.githubusercontent.com/example/social-media-blocklist/main/list.txt"
-      - action: adguard.refresh
+      - service: adguard.refresh

Apply the same corrections to the two other sample blocks (guest filtering & DNS performance).

Also applies to: 168-182, 191-201

source/_integrations/adguard.markdown (1)

204-207: Remove doubled blank line to satisfy markdownlint (MD012)

Markdown style requires a single blank line between sections.

-{% endraw %}
-
-
+{% endraw %}
+

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting