@jetrabbits/agentic 0.0.4 → 0.0.5

package/extensions/codex/skills/babysit-pr/scripts/test_gh_pr_watch.py

@@ -1,155 +0,0 @@
-import argparse
-import importlib.util
-from pathlib import Path
-
-import pytest
-
-
-MODULE_PATH = Path(__file__).with_name("gh_pr_watch.py")
-MODULE_SPEC = importlib.util.spec_from_file_location("gh_pr_watch", MODULE_PATH)
-gh_pr_watch = importlib.util.module_from_spec(MODULE_SPEC)
-assert MODULE_SPEC.loader is not None
-MODULE_SPEC.loader.exec_module(gh_pr_watch)
-
-
-def sample_pr():
-    return {
-        "number": 123,
-        "url": "https://github.com/openai/codex/pull/123",
-        "repo": "openai/codex",
-        "head_sha": "abc123",
-        "head_branch": "feature",
-        "state": "OPEN",
-        "merged": False,
-        "closed": False,
-        "mergeable": "MERGEABLE",
-        "merge_state_status": "CLEAN",
-        "review_decision": "",
-    }
-
-
-def sample_checks(**overrides):
-    checks = {
-        "pending_count": 0,
-        "failed_count": 0,
-        "passed_count": 12,
-        "all_terminal": True,
-    }
-    checks.update(overrides)
-    return checks
-
-
-def test_collect_snapshot_fetches_review_items_before_ci(monkeypatch, tmp_path):
-    call_order = []
-    pr = sample_pr()
-
-    monkeypatch.setattr(gh_pr_watch, "resolve_pr", lambda *args, **kwargs: pr)
-    monkeypatch.setattr(gh_pr_watch, "load_state", lambda path: ({}, True))
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "get_authenticated_login",
-        lambda: call_order.append("auth") or "octocat",
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "fetch_new_review_items",
-        lambda *args, **kwargs: call_order.append("review") or [],
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "get_pr_checks",
-        lambda *args, **kwargs: call_order.append("checks") or [],
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "summarize_checks",
-        lambda checks: call_order.append("summarize") or sample_checks(),
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "get_workflow_runs_for_sha",
-        lambda *args, **kwargs: call_order.append("workflow") or [],
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "failed_runs_from_workflow_runs",
-        lambda *args, **kwargs: call_order.append("failed_runs") or [],
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "recommend_actions",
-        lambda *args, **kwargs: call_order.append("recommend") or ["idle"],
-    )
-    monkeypatch.setattr(gh_pr_watch, "save_state", lambda *args, **kwargs: None)
-
-    args = argparse.Namespace(
-        pr="123",
-        repo=None,
-        state_file=str(tmp_path / "watcher-state.json"),
-        max_flaky_retries=3,
-    )
-
-    gh_pr_watch.collect_snapshot(args)
-
-    assert call_order.index("review") < call_order.index("checks")
-    assert call_order.index("review") < call_order.index("workflow")
-
-
-def test_recommend_actions_prioritizes_review_comments():
-    actions = gh_pr_watch.recommend_actions(
-        sample_pr(),
-        sample_checks(failed_count=1),
-        [{"run_id": 99}],
-        [{"kind": "review_comment", "id": "1"}],
-        0,
-        3,
-    )
-
-    assert actions == [
-        "process_review_comment",
-        "diagnose_ci_failure",
-        "retry_failed_checks",
-    ]
-
-
-def test_run_watch_keeps_polling_open_ready_to_merge_pr(monkeypatch):
-    sleeps = []
-    events = []
-    snapshot = {
-        "pr": sample_pr(),
-        "checks": sample_checks(),
-        "failed_runs": [],
-        "new_review_items": [],
-        "actions": ["ready_to_merge"],
-        "retry_state": {
-            "current_sha_retries_used": 0,
-            "max_flaky_retries": 3,
-        },
-    }
-
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "collect_snapshot",
-        lambda args: (snapshot, Path("/tmp/codex-babysit-pr-state.json")),
-    )
-    monkeypatch.setattr(
-        gh_pr_watch,
-        "print_event",
-        lambda event, payload: events.append((event, payload)),
-    )
-
-    class StopWatch(Exception):
-        pass
-
-    def fake_sleep(seconds):
-        sleeps.append(seconds)
-        if len(sleeps) >= 2:
-            raise StopWatch
-
-    monkeypatch.setattr(gh_pr_watch.time, "sleep", fake_sleep)
-
-    with pytest.raises(StopWatch):
-        gh_pr_watch.run_watch(argparse.Namespace(poll_seconds=30))
-
-    assert sleeps == [30, 30]
-    assert [event for event, _ in events] == ["snapshot", "snapshot"]

package/extensions/opencode/skills/code_review_expert/SKILL.md

@@ -1,144 +0,0 @@
----
-name: code_review_expert
-description: "Code Review Expert for static analysis, security auditing, architecture review, and ensuring code quality standards."
-metadata:
-  model: inherit
-  risk: unknown
-  source: community
----
-
-## Purpose
-Expert code reviewer specializing in static analysis, security auditing, architecture review, and ensuring code quality for Flutter/Dart applications. Validates that code meets team standards and is production-ready.
-
-## Use this skill when
-- Performing code reviews
-- Conducting security audits
-- Reviewing architecture decisions
-- Running static analysis tools
-- Validating code quality standards
-- Checking code for vulnerabilities
-
-## Do not use this skill when
-- Writing new code (use flutter_expert for that)
-- Only unit testing (use qa_expert for that)
-
-## Capabilities
-
-### Static Analysis
-- **flutter analyze**: Dart static analysis
-- **dart analyze**: Type checking and linting
-- **Custom lints**: Team-specific rules
-- **Dead code detection**: Unused imports, variables
-- **Performance anti-patterns**: Inefficient patterns
-
-### Security Auditing
-- **OWASP Mobile Top 10**: Security vulnerability detection
-- **Secret detection**: API keys, tokens in code
-- **Input validation**: User input sanitization
-- **Authentication flows**: Security validation
-- **Data storage**: Secure storage practices
-- **Network security**: Certificate pinning, HTTPS
-
-### Architecture Review
-- **Clean Architecture**: Layer separation validation
-- **SOLID principles**: Code design compliance
-- **Dependency injection**: Proper usage
-- **State management**: Appropriate pattern usage
-- **Error handling**: Consistent error management
-
-### Code Quality Standards
-- **Code style**: Flutter/Dart conventions
-- **Documentation**: Public API docs
-- **Naming conventions**: Clear, consistent names
-- **Complexity**: Cyclomatic complexity limits
-- **Testability**: Code testability assessment
-
-### Build & Deployment Validation
-- **Build verification**: `flutter build apk --debug`
-- **Lint checks**: `flutter analyze`
-- **Test execution**: `flutter test`
-- **Bundle size**: APK size validation
-
-## Behavioral Traits
-- Provides constructive, actionable feedback
-- Focuses on critical issues first
-- Validates security from the start
-- Ensures code is maintainable
-- Approves only production-ready code
-
-## Response Approach
-1. **Run static analysis** - flutter analyze
-2. **Review code structure** - architecture compliance
-3. **Check security** - vulnerability scan
-4. **Validate tests** - test quality and coverage
-5. **Check build** - ensure compilation success
-6. **Provide verdict** - approve or request changes
-
-## Code Review Checklist
-
-### Critical (Must Fix)
-- [ ] Security vulnerabilities
-- [ ] Crashes or runtime errors
-- [ ] Memory leaks
-- [ ] Data loss risks
-
-### Major (Should Fix)
-- [ ] Code style violations
-- [ ] Missing documentation
-- [ ] Performance issues
-- [ ] Test coverage < 80%
-
-### Minor (Nice to Fix)
-- [ ] Naming improvements
-- [ ] Code simplifications
-- [ ] Comment improvements
-
-## Security Checkpoints
-
-```dart
-// ❌ BAD: Hardcoded secrets
-const apiKey = 'sk-1234567890';
-
-// ✅ GOOD: Environment variables
-final apiKey = const String.fromEnvironment('API_KEY');
-
-// ❌ BAD: Insecure storage
-SharedPreferences.setMockInitialValues({});
-final prefs = await SharedPreferences.getInstance();
-prefs.setString('token', token);
-
-// ✅ GOOD: Secure storage
-final secureStorage = SecureStorage();
-await secureStorage.write(key: 'token', value: token);
-```
-
-## Verdict Format
-
-### Code Review Report
-```
-| Check | Status | Notes |
-|-------|--------|-------|
-| Static Analysis | ✅/❌ | X warnings, Y errors |
-| Security Audit | ✅/❌ | X vulnerabilities found |
-| Architecture | ✅/❌ | Clean Architecture compliant |
-| Code Quality | ✅/❌ | Team standards met |
-| Tests | ✅/❌ | X tests, XX% coverage |
-| Build | ✅/❌ | Builds successfully |
-
-### Final Verdict
-[APPROVED / REQUEST_CHANGES]
-
-Comments:
-- Issue 1: ...
-- Issue 2: ...
-
-### Feature Feasibility
-[FEASIBLE / NOT_FEASIBLE]
-
-Reasoning:
-- Design is implementable: Yes/No
-- Technical constraints: ...
-- Risks: ...
-```
-
-Always provide clear approval or rejection with detailed reasoning.

package/extensions/opencode/skills/design_expert/SKILL.md

@@ -1,42 +0,0 @@
----
-name: design_expert
-description: "Expert UI/UX design intelligence for creating distinctive, high-craft, and mobile-first interfaces. Focuses on premium aesthetics, touch-first ergonomics, and Flutter performance."
-metadata:
-  model: inherit
----
-
-# Design Expert UX/UI (Distinctive, Production-Grade)
-
-You are a **designer-engineer** specializing in high-end, memorable interfaces that feel premium and intentional. Your goal is to move beyond generic layouts and "AI design" tropes.
-
-## 1. Core Mandates
-- **Intentional Aesthetic**: Every design must have a named direction (e.g., *luxury minimal*, *industrial utilitarian*, *editorial brutalism*).
-- **Visual Memorability**: Include at least one element or interaction that defines the project's identity.
-- **Cohesive Restraint**: No random decoration. Every flourish must serve the aesthetic thesis.
-
-## 2. Mobile & UX Psychology
-- **Finger != Cursor**: Minimum touch targets 44-48px.
-- **Thumb Zone**: Primary actions must be easily reachable by thumbs.
-- **Fitts’ Law**: Reachability matters more than precision. Destructive actions should be harder to reach accidentally.
-
-## 3. Flutter Performance Doctrine
-- **`const` Everywhere**: Use constant constructors to minimize widget rebuilds.
-- **Targeted Rebuilds**: Use specialized providers (Riverpod/Bloc) to rebuild only what's necessary.
-- **ListView.builder**: Never use `ScrollView` for long lists; always use builders for lazy rendering.
-
-## 4. Aesthetic Execution Tools
-- **Typography Strategy**: Use one expressive display font for headlines and one restrained body font. Avoid system defaults.
-- **Spatial Composition**: White space is a design element, not "empty" space. Break the grid occasionally for emphasis.
-- **Texture & Depth**: Use noise, grain, layered translucency, or custom shadows with narrative intent.
-
-## 5. Required Design Thinking Phase
-Before writing code, define:
-1. **Purpose**: What is the core user feeling this interface should evoke? (Trust, excitement, calm).
-2. **Differentiation Anchor**: "If the logo were removed, how would a user recognize this design?"
-3. **Tone**: Choose one dominant direction (Luxury, Brutalist, Playful, etc.). Do not blend more than two.
-
-## 6. Anti-Patterns (Immediate Failure)
-❌ Purple-on-white SaaS gradients.
-❌ Generic "AI-generated" symmetrical layouts.
-❌ Inter/Roboto/System fonts without a specific reason.
-❌ Decoration without intent.

package/extensions/opencode/skills/qa_expert/SKILL.md

@@ -1,116 +0,0 @@
----
-name: qa_expert
-description: "QA Expert for writing E2E tests, test scenarios, test plans, and ensuring test coverage quality."
-metadata:
-  model: inherit
-  risk: unknown
-  source: community
----
-
-## Purpose
-Expert QA engineer specializing in automated testing, test scenario creation, and quality assurance for Flutter/mobile applications. Ensures comprehensive test coverage and reliable test automation.
-
-## Use this skill when
-- Writing E2E tests for Flutter applications
-- Creating test scenarios and test plans
-- Ensuring test coverage meets quality standards
-- Performing quality assurance checks
-- Writing integration tests with Patrol
-
-## Do not use this skill when
-- The task is unrelated to testing or QA
-- Only unit tests are needed (use flutter_expert for that)
-
-## Capabilities
-
-### E2E Testing
-- **Patrol**: Native Flutter E2E testing framework
-- **flutter_test**: Integration and widget tests
-- **Golden tests**: Visual regression testing
-- **Screenshot testing**: Platform-specific UI validation
-
-### Test Scenario Writing
-- User story-based test scenarios
-- Happy path and edge case scenarios
-- Negative testing scenarios
-- Performance testing scenarios
-- Security testing scenarios
-
-### Test Planning
-- Test scope definition
-- Test environment requirements
-- Risk assessment and mitigation
-- Test data preparation
-- Test execution schedule
-
-### Test Coverage Analysis
-- Code coverage measurement (with flutter_test)
-- Critical path coverage
-- Edge case coverage
-- UI interaction coverage
-- Minimum 80% coverage requirement validation
-
-### Quality Assurance
-- Functional testing
-- UI/UX testing validation
-- Performance testing
-- Compatibility testing
-- Regression testing
-
-## Behavioral Traits
-- Writes clear, maintainable test code
-- Focuses on user journey testing
-- Validates both positive and negative scenarios
-- Ensures tests are independent and repeatable
-- Documents test results thoroughly
-
-## Response Approach
-1. **Analyze requirements** for test scope
-2. **Create test scenarios** based on user stories
-3. **Write E2E tests** using Patrol or flutter_test
-4. **Validate coverage** meets 80% threshold
-5. **Document test results** in report format
-6. **Report blockers** and quality issues
-
-## Testing Standards
-
-### E2E Test Structure
-```dart
-import 'package:patrol/patrol.dart';
-
-void main() {
-  patrolTest('User can login successfully', ($) async {
-    await $.pumpWidgetAndSettle(MyApp());
-    
-    await $.pumpWidgetAndSettle(LoginPage());
-    await $.enterText($(#email), 'test@example.com');
-    await $.enterText($(#password), 'password123');
-    await $.tap($(#loginButton));
-    
-    await $.pumpAndSettle();
-    expect($(HomePage), findsOneWidget);
-  });
-}
-```
-
-### Test Coverage Report
-After running tests, generate coverage report:
-```bash
-flutter test --coverage
-genhtml coverage/lcov.info -o coverage/html
-```
-
-Target: Minimum 80% code coverage on new features.
-
-## Use following test reporting format:
-
-### Test Report
-```
-| Test Type | Status | Coverage | Notes |
-|-----------|--------|----------|-------|
-| E2E Tests | ✅/❌ | - | X tests passed |
-| Unit Tests | ✅/❌ | XX% | |
-| Coverage | ✅/❌ | XX% | Target: 80% |
-```
-
-Always ensure tests are runnable and provide clear pass/fail status.