Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

alias_method doesn't work with ActiveRecord attribute methods;
use a regular method definition instead.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Use ::OpenStruct to avoid NameError in Repository model
- Handle Time objects from Octokit in fetch_starred_since
- Add webmock and stub StarEvent.fetch_and_upsert in tests to
  prevent unintended GitHub API calls during test suite

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Ruby 3.4+ requires explicit ostruct gem as it was removed from
the default gems. Revert ::OpenStruct to OpenStruct now that the
gem is properly declared.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Restore Hash fallback in avatar_image_tag (lost during rebase merge)
- Update notify.text.erb to use event.actor_login instead of event['actor']['login']

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The repo_* keys in the intermediate hash were not DB columns but
appeared alongside DB columns, causing confusion. Refactor so that
fetch_starred_since returns two distinct collections: star_events
(only the star_events table fields) and repos (repository fields).
upsert_repositories now receives repo data directly without needing
the repo_* prefixed keys.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

all_by and by were identical — both delegate to where(actor_login:)
which accepts a single value or an array. Remove the by class method,
rename all_by scope to by, and update the one call site in User.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Replace LIKE-based owner scope with an equality check on the new
repo_owner column (indexed). Populate repo_owner from repo.owner.login
during fetch, and derive it from repo_name in stub_star_event! for tests.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Data.define provides an immutable value object with an explicit
interface, no method_missing overhead, and errors on unknown
attributes. Also removes the ostruct gem dependency since Data
is built into Ruby 3.2+.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- by_name: was used in the old Mongoid-based repository! lookup on
  StarEvent, which was removed during the PostgreSQL migration
- watchers_count: compatibility alias left over from when repositories
  were fetched directly from GitHub API responses; views access
  stargazers_count directly on the ActiveRecord model

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-authored-by: tricknotes <290782+tricknotes@users.noreply.github.com>
Agent-Logs-Url: https://github.com/tricknotes/starseeker/sessions/88dafe1b-34ea-48bd-b7f5-bf70463355de

Include repo_owner column and its index directly in the initial
create_star_events migration, removing the separate
add_repo_owner_to_star_events migration.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Controllers no longer call StarEvent.fetch_and_upsert directly;
GitHub API access is only triggered via the rake task, which is
never executed during the test suite. There are no stub_request
usages, so webmock provides no value.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fetch_and_upsert_per_user now delegates to fetch_and_upsert_graphql,
reducing HTTP round-trips from N_logins to N_logins / GRAPHQL_BATCH_SIZE
while still using each app-user's own token for their independent rate-limit
budget.

fetch_and_upsert_graphql gains an optional fallback_client: parameter so
the REST pagination fallback uses the same token context as the GraphQL
phase instead of falling back to the shared Settings.github_client.

github_client.following returns both Users and Organizations, but the GraphQL
query uses user(login:) which only resolves User accounts.  Filtering by
type == 'User' at the source removes orgs before they reach any fetch path.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The method was a thin wrapper around fetch_and_upsert_graphql.  Call
fetch_and_upsert_graphql directly from the rake task and remove the wrapper.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The transport protocol is an implementation detail and does not belong in
the public interface name.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…clusion

- User#followings: verify that Organization type accounts are excluded
  and only User type accounts are returned
- StarEvent.fetch_and_upsert: verify that private repos are skipped
  in the REST fallback path (fetch_each_page), consistent with the
  GraphQL path behavior

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>