Save to database #13

2026-07-12T21:51:46Z

haylan commented

2026-07-12 21:51:46 +00:00

No description provided.

haylan added 42 commits 2026-07-12 21:51:46 +00:00

chore(todod): proposal plane for sotaged the history 406da07397

fix(di): correct provider service FQCNs in services.yaml 6d6680ca37

Providers were moved into the App\Service\Provider namespace, but
services.yaml still tagged/configured them under the old App\Service
FQCNs, so the _instanceof conditional never matched and the container
failed to compile.

config: cap http_client timeout and max_duration 09c0be8312

Per-request bounds so a single slow/hung provider can't stall the
whole request indefinitely once fetching happens concurrently.

refactor(provider): split fetch() into startFetch()/resolveFetch() 11bec43bbe

Splits the provider contract into a non-blocking startFetch() that
fires the HTTP request(s) and a resolveFetch() that reads the result,
so callers can fire every provider's request before blocking on any
of them. Symfony HttpClient is already async under the hood -
request() doesn't block until you read the response - so this needs
no new dependency.

refactor(github): make GitHubProvider fetch non-blocking 9f3c54afd1

startFetch() fires the GraphQL POST and returns the response without
reading it; resolveFetch() does the .toArray()/parse/log work that
used to happen inline right after the request.

refactor(gitea): make GiteaProvider fetch non-blocking 596260bb85

startFetch() fires the heatmap GET and returns the response without
reading it; resolveFetch() does the .toArray()/parse/log work.

refactor(gitlab): make GitLabProvider fetch non-blocking 9b631f4777

startFetch() resolves the user id and fires the first events page,
returning a handle without reading it; resolveFetch() continues the
existing do/while pagination starting from that first response.

Pagination itself stays sequential within this provider since each
page depends on the previous one - parallelism here only spans across
providers, not within GitLab's own pages.

feat(aggregator): fetch all providers concurrently fcf841f7ad

aggregate() now fires every configured provider's startFetch() in one
pass before resolving any of them, so wall-clock cost is roughly
max(providers) instead of sum(providers). Partial-failure isolation
and per-date summation behavior are unchanged.

test(provider): update ProbeTraitTest stub for split fetch API f2bf24994c

The anonymous ProviderInterface implementation needs startFetch()/
resolveFetch() instead of fetch() to stay loadable.

test(github): exercise startFetch()/resolveFetch() instead of fetch() 5c47888a49

test(gitea): exercise startFetch()/resolveFetch() instead of fetch() c89b4e9212

test(gitlab): exercise startFetch()/resolveFetch() instead of fetch() 1e2f022579

test(aggregator): cover concurrent fetch failure modes 8fb7781227

Mocks startFetch()/resolveFetch() instead of fetch(), and splits the
old single failure test into separate start-throws and
resolve-throws cases since those are now distinct call sites.

chore(controller): lower set_time_limit now that fetching is concurrent cd326a34ba

Providers fetch in parallel now (cost ~= max, not sum), so the old
90s stopgap for the 3-sequential-providers-plus-pagination worst case
can come back down toward the 30s default, with headroom for
pagination and per-request HTTP timeouts.

chore(composer): remove deprecated Guzzle and GraphQL client configurations ca94b80beb

chore(TODO): mark completed tasks and update contribution store details f52a3449c6

fix(provider): dedoupet base URL is properly formatted in GitLab and Gitea providers 5e27a5bc93

chore(docker): update volume mappings for development environment cb89a1c6b9

feat(store): fist stepts for implement ContributionStore for managing contributions in SQLite 3d2c0487b7

feat(db): prepared add and remove function c929fdfda6

fix(provider): normalize baseUrl before assigning readonly property 78e7a72ee9

Promoted readonly properties can't be reassigned in the constructor
body, so GitLabProvider and GiteaProvider threw on every instantiation.
Normalize the plain parameter first, then assign to a separately
declared readonly property.

refactor(github): build GraphQL query without the client bundle 885dec357e

The eightpoints/guzzle-bundle and idci/graphql-client-bundle only
existed to build one near-static query string, and the bundle's own
HTTP transport was already bypassed in favor of Symfony HttpClient.
Build the query with sprintf/json_encode instead, drop both bundles
from config/bundles.php, and update the test double accordingly.

feat(store): finish ContributionStore and add typed entities 9087f91855

- Fix add()'s upsert SQL (was invalid SQL from typos) and use it for
  the new merge() as well.
- Fix remove()'s missing FROM keyword and drop the unindexed LIKE scan
  in favor of an exact match.
- Add latestDate(), all() with optional sinceDays filtering, and
  prune() with a retention-day cutoff.
- Add Contribution (immutable value object) and ContributionCollection
  (IteratorAggregate + Countable) so all() returns something typed
  instead of a raw date => count array.
- Cover all of the above with unit tests against an in-memory SQLite
  database.

fix(docker): target the dev stage when building for local development b3dc7a2298

docker-compose.override.yml only overrode the dockerfile, so it kept
inheriting target: final from docker-compose.yml — but Dockerfile.dev
is single-stage and has no final stage, so the dev build failed. Name
the stage in Dockerfile.dev and target it explicitly from the
override.

docs(readme): fix phpunit command paths bce564d68b

There is no bin/phpunit (only bin/console) — the binary lives at
vendor/bin/phpunit, matching CLAUDE.md's documented commands.

chore(todo): mark completed tasks and expand step 3 detail a76ed6efc8

chore(todo): expanded todo to fix a live bug e2a324b74a

feat(db): wired databas into services config 095f3e0253

chore(doc): documentet all public methods e7f14dfd35

chore(todo): checked php stan instalation and configuration. added document php stan erros as well to update readme docs 0d7cf68771

feat(phpstan): added php stan 91bda21f89

chore(namespace): rewirtten namspeace from app to GitContributionGraph. 41e88144a4

feat(store): fetch contributions incrementally and persist history in SQLite 4ff45b7c46

Bound each provider's startFetch()/resolveFetch() to an optional
since/until window, wire a SQLite-backed ContributionStore into
ContributionAggregator (fetch only the trailing window past the last
stored date, merge into the store, prune by CONTRIBUTIONS_RETENTION_DAYS),
and add a graph:contributions:refetch command to force a full or ranged
re-fetch in <=365-day chunks. This is the root fix for the full
365-day-refetch timeout that used to hit on every cache miss.

fix(renderer): coerce strtotime() result to int before formatting dates 14ea788522

date() expects int|null, but strtotime() returns int|false; cast to make
the type explicit for PHPStan level 8 (the date strings are always
well-formed here, so strtotime() never actually returns false).

fix(entity): re-index ContributionCollection constructor arguments 8e349c961b

array_map() over the variadic constructor can produce non-sequential or
string keys, which doesn't match the array<int, Contribution> property
type; re-index with array_values(). Also documents the iterator/count
accessors.

docs(phpdoc): document public methods across src/ 9c028aaf5e

Add professional PHPDoc (summary + @param/@return) to the remaining
public methods that had none or only partial coverage: GraphController's
actions, ContributionStore's accessors, SvgRenderer's render() and
private helpers (with concrete array-shape annotations), ProbeTrait,
ProviderHealthChecker, ProviderStatus, and the Contribution entity.
Also adds the matching @return shapes on SvgRendererTest's data
providers.

chore(phpstan): raise analyser memory limit and record resolved findings de831beaaa

Add --memory-limit=512M to the phpstan composer script — the default
128M crashed the level-8 run under parallel workers. Update
PHP-Stan-Errors.md to note all 22 catalogued findings are now resolved.

docs(readme): correct outdated deploy docs and docker-compose examples cf7bd8d70c

README had drifted from the codebase: wrong test path, a stale /health
sample response, a docker-compose volume mount pointing at a path the
image never writes to (with no SQLite data volume at all), a missing
CONTRIBUTIONS_RETENTION_DAYS example, and an architecture diagram
predating the SQLite store/refetch command. Also fixes
docker-compose.prod.yml, which was missing the data volume and
CONTRIBUTIONS_RETENTION_DAYS env var present in docker-compose.yml —
without it, a prod deployment on this file loses contribution history
on every container recreate.

docs(claude): document SQLite store, refetch command, and phpstan script 20e943a2b5

Update the architecture diagram and env var table to reflect the
ContributionStore tier and incremental fetch, document the
graph:contributions:refetch command, and note composer phpstan next to
the existing test commands.

chore(todo): mark completed roadmap items be6146d5ad

Check off the incremental-fetch/store wiring, refetch command, phpstan
setup, and CLAUDE.md update items now that all are done.

chore(phpstan): remove PHP-Stan-Errors.md ac515400e0

All findings it catalogued are resolved; drop the file rather than
keep a stale error log around.

chore(update): changelog 0.2.0 4825c45a7c

haylan merged commit c4dffb1dc6 into main

2026-07-12 21:51:57 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: haylan/git-contribution-graph#13