Skip to content

Development#2816

Merged
Armanul46 merged 21 commits intotrunkfrom
development
Apr 15, 2026
Merged

Development#2816
Armanul46 merged 21 commits intotrunkfrom
development

Conversation

@Armanul46
Copy link
Copy Markdown
Collaborator

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

Any linked issues

Fixes #

Checklist

RabbiIslamRony and others added 21 commits February 17, 2026 14:43
- Remove prepare script and lint-staged block from package.json
- Make .husky/pre-commit branch-aware (only runs lint-staged on trunk)
- Add custom merge driver in .gitattributes to protect package.json during merges
Add GitHub Actions workflow that uses anthropics/claude-code-action to
automatically review pull requests. Reviews cover code quality, security,
performance, and WordPress/Directorist coding standards. Also adds CLAUDE.md
with comprehensive project architecture documentation for AI-assisted reviews.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The claude-code-action requires id-token: write permission to fetch
an OIDC token for GitHub authentication.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Switch from pull_request to pull_request_target so fork PRs have
  access to repo secrets and OIDC tokens
- Pass github_token explicitly to skip OIDC token generation
- Checkout PR head SHA explicitly for correct code review context

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
PRs in this repo target the development branch, not trunk.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Build: Add automated Claude Code PR review workflow
…not rendering

wp_kses_post strips <input> tags, causing the checkbox to be silently removed
while the label remained. Extended the allowlist to permit input with safe attributes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Reformatted multi-line wp_kses call to comply with PEAR function call
signature and Generic array indentation standards.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ntation

PHPCS cannot correctly handle multi-line PHP inside a JS template literal.
Moved $allowed_tags computation before the <script> block so the inline
PHP reduces to a single echo statement.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
fix: use wp_kses with input allowlist to fix email verified checkbox …
Fix: CSV import fails to download images from extension-less CDN URLs (e.g. logo.dev)
Copy link
Copy Markdown
Member

@RabbiIslamRony RabbiIslamRony left a comment

Choose a reason for hiding this comment

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

Approved

@Armanul46 Armanul46 merged commit e96b086 into trunk Apr 15, 2026
2 checks passed
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