Skip to content

AI assisted development skill#125

Open
wzieba wants to merge 3 commits into
mainfrom
ai-assisted-development
Open

AI assisted development skill#125
wzieba wants to merge 3 commits into
mainfrom
ai-assisted-development

Conversation

@wzieba
Copy link
Copy Markdown
Collaborator

@wzieba wzieba commented May 21, 2026

No description provided.

wzieba added 2 commits May 21, 2026 12:34
Adds three skills covering setup, tracking, and testing:
- parsely-android-setup: Gradle dependency, Application init, DI wiring
- parsely-android-tracking: pageview, engagement, video, conversion, metadata
- parsely-android-testing: FakeParselyTracker with typed data classes and test patterns
Restructures skills into a Claude Code plugin under plugins/parsely-android/,
adds plugin manifests, and documents installation in the README.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.32%. Comparing base (6c22480) to head (f1fde3f).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #125   +/-   ##
=======================================
  Coverage   71.32%   71.32%           
=======================================
  Files          22       22           
  Lines         415      415           
  Branches       50       50           
=======================================
  Hits          296      296           
  Misses        105      105           
  Partials       14       14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@wzieba
Copy link
Copy Markdown
Collaborator Author

wzieba commented May 21, 2026

Code review

Found 1 issue:

  1. ParselyVideoMetadata constructor parameter durationSeconds is declared internal in the SDK, so using it as a named argument from an external module causes a Kotlin compile error. Any developer following this skill will get error: cannot access 'durationSeconds': it is internal in 'ParselyVideoMetadata'. The argument should be passed positionally as the last parameter.

videoMetadata = ParselyVideoMetadata(
videoId = videoId,
durationSeconds = durationSecs,
title = "My Video Title", // optional but recommended
)
)
}

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

durationSeconds is declared internal, so it cannot be used as a named
argument from external modules. Use positional arguments instead.
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