Skip to content

Resolve analytics video and playlist titles#47

Merged
jdwit merged 1 commit into
mainfrom
issue-45-resolve-analytics-titles
Jun 18, 2026
Merged

Resolve analytics video and playlist titles#47
jdwit merged 1 commit into
mainfrom
issue-45-resolve-analytics-titles

Conversation

@jdwit

@jdwit jdwit commented Jun 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a --resolve flag to analytics query
  • hydrate video/playlist dimensions with batched Data API snippet lookups
  • include videoTitle/playlistTitle in table, JSON, and CSV outputs
  • update generated skill reference

Closes #45

Tests

  • uv run pytest
  • uv run ruff check src tests

@jdwit

jdwit commented Jun 18, 2026

Copy link
Copy Markdown
Owner Author

Subagent review (codex review --base main):

The new --resolve flag is implemented consistently across the command, docs, and tests. I did not find a discrete regression or correctness issue in the diff relative to main.

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.50000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 84.28%. Comparing base (7b9d4a3) to head (e7df5dc).

Files with missing lines Patch % Lines
src/ytstudio/commands/analytics.py 97.50% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   83.90%   84.28%   +0.37%     
==========================================
  Files          15       15              
  Lines        2460     2500      +40     
  Branches      356      367      +11     
==========================================
+ Hits         2064     2107      +43     
+ Misses        264      262       -2     
+ Partials      132      131       -1     

☔ View full report in Codecov by Harness.
📢 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.

@jdwit jdwit force-pushed the issue-45-resolve-analytics-titles branch from dd15993 to e7df5dc Compare June 18, 2026 10:46
@jdwit jdwit merged commit b2488fb into main Jun 18, 2026
8 checks passed
@jdwit jdwit deleted the issue-45-resolve-analytics-titles branch June 18, 2026 10:49
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.

analytics query: resolve video/playlist dimension IDs to titles

1 participant