Skip to content

quality: resolve open remark-lint code-scanning findings and fix codacy workflow formatting - #169

Merged
ThePlenkov merged 3 commits into
mainfrom
fix/main-health
Aug 12, 2026
Merged

quality: resolve open remark-lint code-scanning findings and fix codacy workflow formatting#169
ThePlenkov merged 3 commits into
mainfrom
fix/main-health

Conversation

@ThePlenkov

@ThePlenkov ThePlenkov commented Aug 12, 2026

Copy link
Copy Markdown
Member

Summary

Closes the 45 open remark-lint code-scanning alerts reported by Codacy:

  • list-item-bullet-indent (27 alerts): Converted YAML tags: block lists in .agents/workflows/*.md and .claude/commands/opsx/*.md frontmatter to inline arrays so the indented bullets are no longer parsed as markdown list items.
  • no-undefined-references / no-shortcut-reference-link (18 alerts): Escaped the opening brackets of non-link bracket sequences in CHANGELOG.md (e.g. starts[0], @devin-ai-integration[bot], '\[^']_', \[^<]+) so remark no longer treats them as shortcut/reference links.
  • Also applies prettier formatting to .github/workflows/codacy.yml to fix the current main CI nx format:check failure.

These are root-cause fixes; no suppression comments or global exclusions were added.

Link to Devin session: https://app.devin.ai/sessions/754774d02df644c5894ee8b773029700
Requested by: @ThePlenkov


Summary by cubic

Fixes 45 Codacy remark-lint alerts and unblocks CI by formatting the Codacy workflow file. Converts agent frontmatter tags to inline arrays and escapes non-link brackets in the changelog.

  • Bug Fixes
    • Converted YAML frontmatter tags: lists in .agents/workflows/*.md and .claude/commands/opsx/*.md to inline arrays to clear list-item-bullet-indent.
    • Escaped non-link bracket sequences in CHANGELOG.md to satisfy no-undefined-references and no-shortcut-reference-link.
    • Applied prettier to .github/workflows/codacy.yml to pass nx format:check.

Written for commit 452ab62. Summary will update on new commits.

Review in cubic

devin-ai-integration Bot and others added 3 commits August 12, 2026 08:36
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
…nce rules

Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
Co-Authored-By: Petr Plenkov <petr.plenkov@gmail.com>
@ThePlenkov ThePlenkov self-assigned this Aug 12, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@netlify

netlify Bot commented Aug 12, 2026

Copy link
Copy Markdown

Deploy Preview for adt-cli canceled.

Name Link
🔨 Latest commit 452ab62
🔍 Latest deploy log https://app.netlify.com/projects/adt-cli/deploys/6a7c30bf982bf600085e70f0

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@ThePlenkov, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 11 minutes

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

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a9c61293-e42d-4fb4-8396-af9fd5f74f37

📥 Commits

Reviewing files that changed from the base of the PR and between afdb26d and 452ab62.

📒 Files selected for processing (11)
  • .agents/workflows/lint.md
  • .agents/workflows/opsx-apply.md
  • .agents/workflows/opsx-archive.md
  • .agents/workflows/opsx-explore.md
  • .agents/workflows/opsx-propose.md
  • .claude/commands/opsx/apply.md
  • .claude/commands/opsx/archive.md
  • .claude/commands/opsx/explore.md
  • .claude/commands/opsx/propose.md
  • .github/workflows/codacy.yml
  • CHANGELOG.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.

@amazon-q-developer amazon-q-developer Bot 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.

This PR successfully resolves 45 open remark-lint code-scanning alerts and fixes workflow formatting issues. All changes are formatting corrections that maintain functionality while improving code quality.

Changes Summary:

  • Converted YAML block-style tags: to inline arrays in 9 markdown frontmatter blocks
  • Escaped bracket sequences in CHANGELOG.md to prevent false-positive reference link warnings
  • Applied prettier formatting to .github/workflows/codacy.yml

All fixes are root-cause solutions with no suppressions or exclusions added. The changes are ready to merge.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@nx-cloud

nx-cloud Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit 452ab62

Command Status Duration Result
nx affected -t lint test build e2e-ci --verbose... ✅ Succeeded 46s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-08-12 08:38:46 UTC

@ThePlenkov
ThePlenkov marked this pull request as ready for review August 12, 2026 08:39
Comment thread .agents/workflows/lint.md
Comment thread .agents/workflows/lint.md
Comment thread .agents/workflows/opsx-apply.md
Comment thread .agents/workflows/opsx-apply.md
Comment thread .agents/workflows/opsx-archive.md
Comment thread .claude/commands/opsx/archive.md
Comment thread .claude/commands/opsx/explore.md
Comment thread .claude/commands/opsx/explore.md
Comment thread .claude/commands/opsx/propose.md
Comment thread .claude/commands/opsx/propose.md
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@baz-reviewer

baz-reviewer Bot commented Aug 12, 2026

Copy link
Copy Markdown

Merger

Ready to Merge

MERGE: low-risk lint/documentation and workflow-formatting changes match the stated intent, with all discussions settled and no concrete blocker present. CI was not recorded, but the diff is clearly trivial and contains no behavioral code changes.

Commit 452ab62 · Evaluated 2026-08-12 08:42 UTC

Review this PR on Baz | Customize your next review

@ThePlenkov
ThePlenkov merged commit f7948ab into main Aug 12, 2026
28 checks passed
@ThePlenkov
ThePlenkov deleted the fix/main-health branch August 12, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants