Skip to content

[PAR-805] Add Claude configuration and capability definitions#20

Merged
mescalantea merged 4 commits into
masterfrom
chore/PAR-805-Add-Claude-configuration-and-capability-definitions
Jun 17, 2026
Merged

[PAR-805] Add Claude configuration and capability definitions#20
mescalantea merged 4 commits into
masterfrom
chore/PAR-805-Add-Claude-configuration-and-capability-definitions

Conversation

@mescalantea

@mescalantea mescalantea commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

What is the goal?

Bring this repo in line with the Claude Code tooling added across seQura repos under PAR-805 (integration-core#63, magento2-core#116, woocommerce-sequra#164), adapted to this repo as a published Composer library. Adds a tracked permission policy and keeps the Claude tooling out of the distributed package.

References

How is it being implemented?

  • .claude/settings.json — tracked team permission policy scoped to this repo's real tooling (no bin/ wrappers like the plugin repos): allows vendor/bin/phpunit, composer, php -l, read-only git, and the sq-git:* skills; denies reading .env*/*.pem/*.key; asks before git push.
  • .gitattributes — adds export-ignore for /CLAUDE.md and /.claude/ so the Claude tooling is excluded from the package installed via Composer (git archive), consistent with the existing dev-file exclusions.

Personal prefs stay in the untracked settings.local.json.

Caveats

Scope is intentionally minimal vs. the sibling PRs. Git hooks / setup.sh were omitted: this repo has no QA CI to mirror (only packagist.yml), no phpcs/phpstan config, and the PHPUnit suite needs a live MySQL DB so it can't run unattended in a hook. A version-bump skill was also omitted — the library has no version in composer.json (Packagist uses git tags).

How is it tested?

.claude/settings.json validated as JSON; .gitattributes export-ignore confirmed via git archive (Claude tooling excluded from the published artifact).

How is it going to be deployed?

Standard deployment. Developer-only tooling — not part of the distributed Composer package.

mescalantea and others added 3 commits June 17, 2026 09:14
…ing files

Add .claude/settings.json with a team permission policy scoped to this
repo's tooling (vendor/bin/phpunit, composer, read-only git, sq-git
skills; deny .env/keys; ask before git push), and export-ignore CLAUDE.md
and .claude/ so the Claude tooling is excluded from the published package.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mescalantea mescalantea force-pushed the chore/PAR-805-Add-Claude-configuration-and-capability-definitions branch from b2e54d6 to cec395c Compare June 17, 2026 09:36
@mescalantea mescalantea requested a review from m1k3lm June 17, 2026 09:41
@mescalantea mescalantea marked this pull request as ready for review June 17, 2026 09:41
@mescalantea mescalantea requested a review from a team as a code owner June 17, 2026 09:41

@m1k3lm m1k3lm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approving — tooling/docs-only, low risk. Verified every class CLAUDE.md references exists at its documented path. Two non-blocking nits left inline (a doc version mismatch and a permission-pattern consistency tweak); happy to push the fix if you'd rather not.

Comment thread CLAUDE.md
Comment thread .claude/settings.json Outdated
Aligns the Bash(git check-attr) permission rule with the :* prefix-match
convention used by all other rules in .claude/settings.json.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mescalantea mescalantea merged commit ea1550d into master Jun 17, 2026
4 checks passed
@mescalantea mescalantea deleted the chore/PAR-805-Add-Claude-configuration-and-capability-definitions branch June 17, 2026 11:07
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