Skip to content

Add paperless_ngx docs #39103

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: next
Choose a base branch
from

Conversation

fvgarrel
Copy link

@fvgarrel fvgarrel commented May 19, 2025

Proposed change

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Summary by CodeRabbit

  • Documentation
    • Added new documentation for integrating Paperless-ngx with Home Assistant, including setup instructions, required permissions, configuration details, and available sensors for monitoring service status and activity.

@home-assistant home-assistant bot added has-parent This PR has a parent PR in a other repo next This PR goes into the next branch labels May 19, 2025
Copy link

netlify bot commented May 19, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit ba13f74
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/682f0496fde4850008685a56
😎 Deploy Preview https://deploy-preview-39103--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

coderabbitai bot commented May 19, 2025

📝 Walkthrough

Walkthrough

A new documentation file for the Paperless-ngx integration has been added. This file provides setup instructions, configuration details, and sensor information for connecting Paperless-ngx to Home Assistant, including authentication requirements and removal procedures.

Changes

File(s) Change Summary
source/_integrations/paperless_ngx.markdown Added documentation for the Paperless-ngx integration, including setup, configuration, sensors, and removal instructions.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant HomeAssistant
    participant PaperlessNGX

    User->>PaperlessNGX: Create API access token
    User->>HomeAssistant: Configure Paperless-ngx integration (host, token)
    HomeAssistant->>PaperlessNGX: Authenticate using API token
    PaperlessNGX-->>HomeAssistant: Return status and metrics
    HomeAssistant-->>User: Display Paperless-ngx sensors and status
Loading

Note

⚡️ AI Code Reviews for VS Code, Cursor, Windsurf

CodeRabbit now has a plugin for VS Code, Cursor and Windsurf. This brings AI code reviews directly in the code editor. Each commit is reviewed immediately, finding bugs before the PR is raised. Seamless context handoff to your AI code agent ensures that you can easily incorporate review feedback.
Learn more here.


Note

⚡️ Faster reviews with caching

CodeRabbit now supports caching for code and dependencies, helping speed up reviews. This means quicker feedback, reduced wait times, and a smoother review experience overall. Cached data is encrypted and stored securely. This feature will be automatically enabled for all accounts on May 30th. To opt out, configure Review - Disable Cache at either the organization or repository level. If you prefer to disable all data retention across your organization, simply turn off the Data Retention setting under your Organization Settings.
Enjoy the performance boost—your workflow just got faster.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai or @coderabbitai title anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (2)
source/_integrations/paperless_ngx.markdown (2)

2-3: Fix verb usage in description

The description uses "setup" as a verb; it should be "set up" for correct usage.
Apply this diff:

--- description: Instructions on how to setup the Paperless-ngx integration
+++ description: Instructions on how to set up the Paperless-ngx integration

36-38: Remove extra blank line

There are multiple consecutive blank lines before the config flow include, which may trigger markdownlint errors (MD012). Reduce them to a single blank line.
Apply this diff:

- 
- 
+ 
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between eb88953 and ae4b6d4.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/paperless_ngx.markdown

[uncategorized] ~52-~52: The grammatical number of this noun doesn’t look right. Consider replacing it.
Context: ...ngx: - Number of documents - Number of document with inbox tags - Status of the databas...

(AI_EN_LECTOR_REPLACEMENT_NOUN_NUMBER)

🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: Redirect rules - home-assistant-docs
  • GitHub Check: Header rules - home-assistant-docs
  • GitHub Check: Pages changed - home-assistant-docs
🔇 Additional comments (5)
source/_integrations/paperless_ngx.markdown (5)

1-17: Frontmatter is well-structured

The YAML frontmatter follows Home Assistant guidelines and correctly defines metadata such as ha_release, ha_category, and ha_codeowners.


21-26: Prerequisites section is clear

The permissions requirements for read and admin access are well-documented and adhere to standards.


38-38: Config flow include is correct

The {% include integrations/config_flow.md %} directive properly injects the integration setup instructions.


40-45: Basic configuration block is properly defined

The configuration_basic block clearly specifies the required host and access token parameters.


61-65: Removal instructions are sufficient

The removal section follows the standard pattern, and the {% include integrations/remove_device_service.md %} is correctly referenced.

@c0ffeeca7 c0ffeeca7 added the new-integration This PR adds documentation for a new Home Assistant integration label May 20, 2025
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

♻️ Duplicate comments (1)
source/_integrations/paperless_ngx.markdown (1)

51-59: Standardize sensor list items
The sensor bullet list has inconsistent phrasing and grammatical issues.

Apply this diff to harmonize and pluralize items:

- - Number of documents
- - Number of document with inbox tags
- - Status of the database with errors
- - Status of Redis with errors
- - Status of Celery
- - Status of classifier with errors
- - Status of index with errors
- - Storage Total
- - Storage Available
+ - Number of documents
+ - Number of documents with inbox tags
+ - Database status
+ - Redis status
+ - Celery status
+ - Classifier status
+ - Index status
+ - Total storage
+ - Available storage
🧰 Tools
🪛 LanguageTool

[uncategorized] ~52-~52: The grammatical number of this noun doesn’t look right. Consider replacing it.
Context: ...ngx: - Number of documents - Number of document with inbox tags - Status of the databas...

(AI_EN_LECTOR_REPLACEMENT_NOUN_NUMBER)

🧹 Nitpick comments (2)
source/_integrations/paperless_ngx.markdown (2)

3-3: Fix verb usage in description
The phrase "how to setup" uses the noun "setup" instead of the verb "set up." Please update for grammatical accuracy.

- description: Instructions on how to setup the Paperless-ngx integration
+ description: Instructions on how to set up the Paperless-ngx integration

36-37: Remove extra blank lines
Two consecutive blank lines appear after {% enddetails %}, violating markdown style (MD012). Consolidate to a single blank line.

@@ -35,3 +35,2 @@ {% enddetails %}
-
-
+ 
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between ae4b6d4 and 0cb203e.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_integrations/paperless_ngx.markdown

[uncategorized] ~52-~52: The grammatical number of this noun doesn’t look right. Consider replacing it.
Context: ...ngx: - Number of documents - Number of document with inbox tags - Status of the databas...

(AI_EN_LECTOR_REPLACEMENT_NOUN_NUMBER)

🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: Redirect rules - home-assistant-docs
  • GitHub Check: Header rules - home-assistant-docs
  • GitHub Check: Pages changed - home-assistant-docs
🔇 Additional comments (8)
source/_integrations/paperless_ngx.markdown (8)

1-2: Frontmatter metadata is well-structured
The YAML frontmatter fields correctly define the integration metadata.


4-17: Frontmatter metadata after description looks good
Fields such as ha_release, ha_domain, and related links are correctly specified.


19-20: Introduction is clear and concise
The overview correctly describes the integration’s purpose and usage.


21-26: Prerequisites section is informative
Requirements for read and administrator permissions are clearly stated.


27-35: Access token creation steps are well-written
The step-by-step instructions under the details block are clear and actionable.


38-45: Configuration flow include and basic config snippet
Inclusion of config_flow.md and use of the configuration_basic macro align with Home Assistant documentation standards.


47-49: Sensors header is properly defined
Section title and introductory sentence follow Home Assistant conventions.


61-65: Removing integration instructions are correct
This section follows the standard Home Assistant removal procedure without extra steps.

@c0ffeeca7
Copy link
Contributor

I'm drafting this PR, as changes are needed. Please undraft it, once the changes have been implemented and the PR is ready for review again.

@c0ffeeca7 c0ffeeca7 marked this pull request as draft May 20, 2025 05:48
@fvgarrel fvgarrel marked this pull request as ready for review May 20, 2025 18:07
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (2)
source/_integrations/paperless_ngx.markdown (2)

37-38: Remove extra blank lines to satisfy MD012
There are two consecutive blank lines before the config_flow include. Reduce to a single blank line to adhere to markdownlint rules.

Apply this diff:

  {% enddetails %}
-
-
  {% include integrations/config_flow.md %}
+ 
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


56-57: Remove extra blank lines to satisfy MD012
There are two consecutive blank lines after the sensors table. Please leave only one blank line before the next section header.

Apply this diff:

 | **Firmware** | Displays the installed and latest version of the Paperless-ngx instance. |
-
-
 ## Removing the integration
+ 
📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 0cb203e and 2dbe175.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


55-55: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (3)
source/_integrations/paperless_ngx.markdown (3)

35-38: Remove extra blank lines for consistency
The static analysis tool flagged multiple consecutive blank lines between the end of the details block and the config flow include (MD012). Consolidate to a single blank line to maintain readability.

-{% enddetails %}
-
-
+{% enddetails %}
 
 {% include integrations/config_flow.md %}
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


30-33: Clarify token generation step
Simplify the wording in step 4 to remove directional details and make the instruction more concise.

-4. Under **API Auth Token**, select the right **Refresh** button next to the textbox to generate a new token. Confirm with **yes**.
+4. Under **API Auth Token**, click the **Refresh** button to generate a new token, then confirm.

53-57: Consolidate blank lines around the sensors table
There are two blank lines between the table and the “Removing the integration” section. Reduce to a single blank line for a cleaner layout.

| **Firmware** | Displays the installed and latest version of the Paperless-ngx instance. |
-
-
 ## Removing the integration
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

55-55: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 2dbe175 and 3bb710e.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


55-55: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: Redirect rules - home-assistant-docs
  • GitHub Check: Header rules - home-assistant-docs
  • GitHub Check: Pages changed - home-assistant-docs

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

♻️ Duplicate comments (1)
source/_integrations/paperless_ngx.markdown (1)

40-45: Use lowercase snake_case for configuration keys
Configuration keys in Home Assistant docs should be lowercase and snake_case to match integration parameters.

 {% configuration_basic %}
-Host:
+host:
   description: "The host to connect to"
-API key:
-  description: "The API key for the Paperless-ngx user."
+access_token:
+  description: "The API key for the Paperless-ngx user."
 {% endconfiguration_basic %}
🧹 Nitpick comments (3)
source/_integrations/paperless_ngx.markdown (3)

36-37: Remove extra blank lines for markdown style consistency
There are two consecutive blank lines between the end of the details block and the include. Reduce to a single blank line to satisfy MD012 (no-multiple-blanks).

🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


55-55: Add missing period to sensor description
The description for Tags total is missing a trailing period.

-| **Tags total**           | Indicates the total number of defined tags                               |
+| **Tags total**           | Indicates the total number of defined tags.                              |

60-61: Remove extra blank lines before the next section
Lines 60–61 are two blank lines preceding the “## Removing the integration” heading. Collapse to a single blank line to comply with markdownlint’s MD012 rule.

🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

60-60: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


61-61: Multiple consecutive blank lines
Expected: 1; Actual: 3

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 3bb710e and c0a8d39.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


60-60: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


61-61: Multiple consecutive blank lines
Expected: 1; Actual: 3

(MD012, no-multiple-blanks)

⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: Redirect rules - home-assistant-docs
  • GitHub Check: Header rules - home-assistant-docs
  • GitHub Check: Pages changed - home-assistant-docs
🔇 Additional comments (1)
source/_integrations/paperless_ngx.markdown (1)

7-7: Verify ha_iot_class value against Home Assistant standards
The value Local Pull may not match the officially supported IoT class identifiers (e.g., local_poll, local_push). Please confirm and adjust to a valid, lowercase snake_case identifier if needed.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

♻️ Duplicate comments (1)
source/_integrations/paperless_ngx.markdown (1)

40-45: 🛠️ Refactor suggestion

Standardize configuration keys to lowercase snake_case
Configuration keys must match the integration’s parameters and follow Home Assistant’s snake_case convention. Rename Hosthost and API keyaccess_token, and update the description accordingly.

 {% configuration_basic %}
-Host:
-  description: "The host to connect to"
-API key:
-  description: "The API key for the Paperless-ngx user."
+host:
+  description: "The host to connect to"
+access_token:
+  description: "The access token for the Paperless-ngx user."
 {% endconfiguration_basic %}
🧹 Nitpick comments (3)
source/_integrations/paperless_ngx.markdown (3)

36-38: Remove extra blank lines between the details block and config flow include
There are two consecutive blank lines before the {% include integrations/config_flow.md %} directive. Collapse to a single blank line to satisfy markdownlint’s MD012 rule and improve readability.

-{% enddetails %}
- 
- 
-{% include integrations/config_flow.md %}
+{% enddetails %}
+
+{% include integrations/config_flow.md %}
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


53-58: Normalize sensor names’ capitalization & add missing punctuation
Table entries should use consistent Title Case for sensor names, and descriptions should end with a period.

-| **Document types**       | Indicates the total number of defined document types.                    |
+| **Document Types**       | Indicates the total number of defined document types.                    |
-| **Documents in inbox**   | Indicates the number of documents currently in the inbox.                |
+| **Documents in Inbox**   | Indicates the number of documents currently in the inbox.                |
-| **Tags**                 | Indicates the total number of defined tags                               |
+| **Tags**                 | Indicates the total number of defined tags.                              |

59-62: Remove extra blank line before the “Removing the integration” section
There are two blank lines separating the sensors table and the next header. Collapse to a single blank line to satisfy markdownlint’s MD012 rule.

-| **Total documents**      | Indicates the total number of documents stored.                          |
- 
-## Removing the integration
+| **Total documents**      | Indicates the total number of documents stored.                          |
+## Removing the integration
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

60-60: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between c0a8d39 and 3666ace.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

37-37: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)


60-60: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: Redirect rules - home-assistant-docs
  • GitHub Check: Header rules - home-assistant-docs
  • GitHub Check: Pages changed - home-assistant-docs

ha_release: 2025.7
ha_category:
- Sensor
ha_iot_class: Local Pull
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Use the correct IoT class constant
Home Assistant IoT class values must be lowercase snake_case. Replace the human-readable "Local Pull" with the appropriate constant local_poll to ensure consistency with other integrations and proper rendering.

- ha_iot_class: Local Pull
+ ha_iot_class: local_poll
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
ha_iot_class: Local Pull
ha_iot_class: local_poll
🤖 Prompt for AI Agents
In source/_integrations/paperless_ngx.markdown at line 7, the IoT class value
"Local Pull" is incorrectly capitalized and spaced. Replace this human-readable
string with the lowercase snake_case constant "local_poll" to conform to Home
Assistant's IoT class naming conventions and maintain consistency across
integrations.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

♻️ Duplicate comments (2)
source/_integrations/paperless_ngx.markdown (2)

39-44: Standardize and correct configuration keys
Home Assistant configuration keys should be lowercase snake_case and match the integration parameters. Rename Host to host and API key to access_token, and update the description accordingly.
Apply this diff:

 {% configuration_basic %}
-Host:
-  description: "The host to connect to"
-API key:
-  description: "The API key for the Paperless-ngx user."
+host:
+  description: "The host to connect to"
+access_token:
+  description: "The API auth token for the Paperless-ngx user."
 {% endconfiguration_basic %}

7-7: 🛠️ Refactor suggestion

Use correct IoT class constant
The ha_iot_class value should use the lowercase snake_case constant local_poll instead of the human-readable "Local Polling" to adhere to Home Assistant conventions.
Apply this diff:

-ha_iot_class: Local Polling
+ha_iot_class: local_poll
🧹 Nitpick comments (2)
source/_integrations/paperless_ngx.markdown (2)

55-55: Ensure consistent punctuation in sensor table
Add a missing period at the end of the description for the Tags sensor to match the other entries.

-| **Tags**                 | Indicates the total number of defined tags
+| **Tags**                 | Indicates the total number of defined tags.

59-60: Remove extra blank line
There are multiple consecutive blank lines before the removal section. Reduce to a single blank line to satisfy markdown lint rules (MD012).

- 
- 
+ 
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

59-59: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 3666ace and 945e809.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

59-59: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: Redirect rules - home-assistant-docs
  • GitHub Check: Header rules - home-assistant-docs
  • GitHub Check: Pages changed - home-assistant-docs

@home-assistant home-assistant bot added the parent-merged The parent PR has been merged already label May 22, 2025
@home-assistant home-assistant bot marked this pull request as draft May 22, 2025 10:18
@home-assistant
Copy link

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@fvgarrel fvgarrel marked this pull request as ready for review May 22, 2025 11:47
@home-assistant home-assistant bot requested a review from joostlek May 22, 2025 11:47
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

♻️ Duplicate comments (3)
source/_integrations/paperless_ngx.markdown (3)

7-7: Use the correct IoT class constant.

Home Assistant IoT class values should be lowercase snake_case. Replace:

- ha_iot_class: Local Polling
+ ha_iot_class: local_poll

37-42: Standardize configuration key names to snake_case.

Configuration keys must be lowercase snake_case to match the integration’s parameters. Update this block:

 {% configuration_basic %}
-URL:
-  description: "URL to connect to the Paperless-ngx instance."
-API key:
-  description: "API key to connect to the Paperless-ngx API."
+host:
+  description: "The host URL of your Paperless-ngx instance."
+access_token:
+  description: "The API access token for the Paperless-ngx user."
 {% endconfiguration_basic %}

48-56: Ensure consistent sensor naming and descriptions.

The sensor table entries should follow a uniform naming convention and punctuation. For example:

  • Use either all noun phrases (e.g., “Total documents”) or all gerund phrases (e.g., “Number of documents”).
  • Ensure every description ends with a period.
    Please harmonize these for clarity.
🧹 Nitpick comments (1)
source/_integrations/paperless_ngx.markdown (1)

56-60: Remove extra blank line.

There are multiple consecutive blank lines before the “Removing the integration” section. Condense to a single blank line to satisfy markdownlint rule MD012.

🧰 Tools
🪛 markdownlint-cli2 (0.17.2)

57-57: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

📜 Review details

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

📥 Commits

Reviewing files that changed from the base of the PR and between 945e809 and ba13f74.

📒 Files selected for processing (1)
  • source/_integrations/paperless_ngx.markdown (1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
source/_integrations/paperless_ngx.markdown

57-57: Multiple consecutive blank lines
Expected: 1; Actual: 2

(MD012, no-multiple-blanks)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-parent This PR has a parent PR in a other repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch parent-merged The parent PR has been merged already
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy