Skip to content

fix: add GA page-performance compatibility helper#26

Open
Gujiassh wants to merge 1 commit intoTheCraigHewitt:mainfrom
Gujiassh:fix/ga-page-performance
Open

fix: add GA page-performance compatibility helper#26
Gujiassh wants to merge 1 commit intoTheCraigHewitt:mainfrom
Gujiassh:fix/ga-page-performance

Conversation

@Gujiassh
Copy link
Copy Markdown

Summary

  • add GoogleAnalytics.get_page_performance() as the narrow compatibility surface the quick-wins workflow already expects
  • alias avg_engagement_time to the existing averageSessionDuration field in get_top_pages() so downstream callers receive the documented key
  • add a no-network unit test proving exact-path lookup and empty-result behavior

Validation

  • python3 -m py_compile data_sources/modules/google_analytics.py tests/test_google_analytics_compat.py
  • python3 -m unittest tests.test_google_analytics_compat

Notes

  • this is intentionally a provider-layer compatibility fix; it avoids rewriting the existing quick-wins / performance scripts and restores the contract they already call

Restore the GA4 handoff expected by the quick-wins workflow by exposing page-level metrics in the shape the research pipeline already consumes.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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