Skip to content

chore(hooks): add Claude Code hook guard, scrub legacy hooksPath refs#509

Merged
wphillipmoore merged 2 commits into
developfrom
feature/508-hook-guard
May 26, 2026
Merged

chore(hooks): add Claude Code hook guard, scrub legacy hooksPath refs#509
wphillipmoore merged 2 commits into
developfrom
feature/508-hook-guard

Conversation

@wphillipmoore

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

  • Add PreToolUse hook guard, update settings.json, scrub legacy core.hooksPath references from docs

Issue Linkage

Notes

Add .claude/hooks/guard.sh PreToolUse hook, update settings.json with hook wiring and permissions, update docs.
@wphillipmoore wphillipmoore merged commit 99b2d72 into develop May 26, 2026
24 checks passed
@wphillipmoore wphillipmoore deleted the feature/508-hook-guard branch May 26, 2026 19:34
wphillipmoore added a commit that referenced this pull request Jun 2, 2026
…511)

* chore(tooling): refresh managed config to current Vergil tooling/actions

Items 1/3/6/7 of the fleet refresh (epic mq-rest-admin-project/.github#14):
marketplace -> vergil-claude-plugin; embed canonical CLAUDE.md template
(+ vrg-docker-run -> vrg-container-run); ignore .vergil/; cd.yml release
uses secrets: inherit (fixes CD startup_failure). Hook guard (item 2)
already landed via #509. audit COMPLIANT; vrg-validate passes. Refs #510.

* fix(security): source test and example credentials from the environment

Updated CodeQL/Semgrep rulesets flag hard-coded credentials. Remove all
hard-coded password literals from test and example code:

- tests/pymqrest/{test_auth,test_session,test_ensure,test_sync}.py:
  TEST_PASSWORD now reads MQ_TEST_PASSWORD from the env (defaults empty);
  mock transports ignore the value and assertions compare against the
  same constant.
- examples/*.py __main__ blocks: require MQ_ADMIN_PASSWORD from the env
  instead of defaulting to a hard-coded "mqadmin".

vrg-validate (incl. 100% coverage) passes. Refs #510.

* fix(security): use verified SSL context in archived extraction scripts

Semgrep python.lang.security.unverified-ssl-context flagged
ssl._create_unverified_context() in the archived MQSC doc-extraction
scripts. They fetch public IBM docs (https://www.ibm.com/docs, valid
certs), so switch to ssl.create_default_context() — verification works
and the insecure context is removed. Scripts are archived/not run in CI;
change is static-clean for the scanner and more correct if re-run.

Refs #510.

---------

Co-authored-by: Phillip Moore <w.phillip.moore@gmail.com>
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.

1 participant