[bassilkhilo-ag2]

Summary

Adds behavioral tests for 15 skills that had zero test coverage, and wires them into npm test via an extended test:py glob.

Tests added (16 commits, ~400 tests)

Skill / module	Test file	Tests
skills/regression-search/find-regression.py	find-regression.test.py	30
skills/regression-search/diagnose-call.py	diagnose-call.test.py	42
skills/quota-tracker/read-quota.py	read-quota.test.py	19
skills/call-diagnostics/diagnose.py	diagnose.test.py	43
skills/cross-node-sync/regenerate-memory-index.py	regenerate-memory-index.test.py	23
skills/make-viral-video/validate_asset.py	validate_asset.test.py	21
skills/make-viral-video/source_tag.py	test_source_tag.test.py	9
skills/make-viral-video/asset_cache.py	asset_cache.test.py	18
skills/make-viral-video/verify_video.py	verify_video.test.py	27 (approx)
skills/submit-use-case/submit_use_case.py	submit_use_case.test.py	29
skills/agent-registry/registry-client.py	registry-client.test.py	13
skills/agent-registry/registry-service.py	registry-service.test.py	40
skills/deal-finder/scan.py	scan.test.py	40
skills/discord-voice/share_screen_modal.py	share_screen_modal.test.py	~10
skills/image-generation/generate.py	generate.test.py	~10
skills/screen-record/subtitle.py	subtitle.test.py	~25
skills/screen-record/record.py	record.test.py	~25
skills/macos-tools/calendar_reader.py	calendar_reader.test.py	~15
skills/macos-tools/email_sender.py	email_sender.test.py	~15
skills/macos-tools/contacts.py	contacts.test.py	~10
skills/macos-tools/reminders.py	reminders.test.py	~10
skills/tests/publish.py	publish.test.py	14

package.json change

Extends test:py glob from tests/*.test.py → tests/*.test.py skills/tests/*.test.py skills/*/tests/*.test.py so all skill tests are discovered by npm test. Adds test:sh for future shell test files. Without this change the skill tests exist but npm test never runs them.

All tests are pure unit tests — no network calls, no filesystem side-effects, no external dependencies beyond stdlib.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

Bassil Khilo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[VasiliyRad]

Solid infrastructure expansion. The test:py glob now covers tests/, skills/tests/, and skills/*/tests/ — single npm test exercises everything. [ -f "" ] && ... correctly skips unmatched globs. test:sh slot reserved for shell tests. Spot-checked registry-client.test.py — proper importlib.util.spec_from_file_location for hyphenated filenames, env-var-driven workspace, surgical mocks of os.kill for pid_alive edges. Quality looks consistent.

This is 22 new test files / 4560 lines across 15 skills — too much to review line-by-line. Question: does npm test pass green on your machine right now? If yes, ship it; if there's any flake or env dep, please call it out so we know what to expect on first CI run.

Risk note: once this lands, every CI run iterates 22 more test files. Any test that depends on host state (network, specific dirs, env vars beyond what the test sets) becomes a recurring failure. The skip-on-glob-miss pattern means we won't notice if a test file is accidentally deleted, only if a test asserts wrong. Worth a future follow-up to add a "counted test files" assertion ("at least N skills have tests") so we can't silently lose coverage.

23 files, 1 production change (package.json scripts entry). LGTM 👍