Skip to content

Point the HomeAutomation-Config Audit's Escalation at Its Issue - #1785

Merged
ptr727 merged 1 commit into
developfrom
fix/ha-audit-escalation-1783
Sep 24, 2026
Merged

ptr727 merged 1 commit into
developfrom
fix/ha-audit-escalation-1783

Conversation

@ptr727

@ptr727 ptr727 commented Sep 24, 2026

Copy link
Copy Markdown
Owner

The HomeAutomation-Config audit report said its workflow-token escalation was "Not yet filed". That escalation has since been filed as #1783, so the report now says "Filed as hub #1783", matching how its sibling escalations cite hub #727 and hub #1134.

Raised by Copilot on promotion PR #1784. A fix to develop's content can't be pushed to the promotion PR itself, so it lands here first.

🤖 Generated with Claude Code

The workflow-token escalation said it was not yet filed. It is now
tracked as hub #1783.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 24, 2026 21:30
@ptr727

ptr727 commented Sep 24, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 024a3abc-a1f0-42d9-808c-7dfcb808bcac

📥 Commits

Reviewing files that changed from the base of the PR and between ecdabb6 and 3ca5ac8.

📒 Files selected for processing (1)
  • reports/homeautomation-config/audit.md

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

Comment @coderabbitai help to get the list of available commands.

@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The change is a straightforward, internally consistent report update that matches the filed escalation issue.

Review effort: Lite
Findings: None

What changed in this PR

Updates the HomeAutomation-Config audit report to point the workflow-token escalation at the now-filed hub issue, aligning this report with the existing escalation cross-references.

Changes:

  • Replace the escalation status text from "Not yet filed" to "Filed as hub #1783" in the HomeAutomation-Config audit report.
File Description
reports/​homeautomation-config/​audit.md Updates the escalation entry to reference hub issue #1783 now that it exists.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ptr727
ptr727 merged commit 21917a0 into develop Sep 24, 2026
9 checks passed
@ptr727
ptr727 deleted the fix/ha-audit-escalation-1783 branch September 24, 2026 21:36
ptr727 added a commit that referenced this pull request Sep 24, 2026
…1784)

Promotes the rewritten HomeAutomation-Config audit report, which now
grades that repository on the release workflow model, and three
corrections to it that Copilot raised on this PR.

## What reaches main

- **Report (#1781):** `reports/homeautomation-config/audit.md` is re-run
against HomeAutomation-Config's `main` after its resync and promotion.
- **Verdict:** not operational. The mechanical run is clean, and two
hand-judged `linter-parity` defects fail it.
- **Drift:** the byte-identical intent advisories (#727), the
`PSScriptAnalyzerSettings.psd1` path collision (#1718), and two callers
without `permissions: {}`.
- **Escalation (#1785):** the workflow-token escalation now points at
its issue, hub #1783, rather than saying it is not yet filed.
- **Reference (#1786):** "promoted in #1768" is now qualified as hub
#1768.
- **Intent advisories (#1787):** the report now says what the advisory
measures, each file's `intentRef` document rather than the hub's own
copy. That is why a byte-identical file still gets flagged.

No hub issue closes with this promotion. #727, #1718, and #1783 stay
open.

## Review state

- **Merge:** merged to `develop` as ecdabb6 (#1781), 21917a0 (#1785),
e4fc02a (#1786), and b7fb7a4 (#1787).
- **Copilot:** four rounds on this PR, the last on head b7fb7a4 with
nothing new. Its one thread was fixed and resolved. Both
previously-missed findings are answered in the conversation, each with
its fixing commit.
- **CodeRabbit:** reviewed head ecdabb6 with no actionable comments,
then was rate-limited for the three one-line fix PRs. Copilot reviewed
each of those on its own head with no findings.
- **Local strict review:** a recorded pass on each fix PR's pushed head.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants