93 api rss feed tbv status updates#124
Conversation
|
✅ Changie-fragment gevonden. |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
…handling[deploy-test]
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
There was a problem hiding this comment.
Pull request overview
This PR adds an RSS 2.0 feed endpoint per API that exposes “content change” events, backed by a new persisted feed-events table. Events are recorded when lifecycle fields change, ADR score changes, the OpenAPI hash changes, or when the OpenAPI document becomes unreachable during the daily refresh job.
Changes:
- Add
/apis/{id}/feedpublic endpoint that returns an RSS feed (application/rss+xml) with an Atom self link. - Persist feed events via new
ApiFeedEventmodel + repository methods, and record events from lifecycle updates, lint updates (ADR score / OAS hash), and refresh failures (OAS unreachable). - Add URL helper utilities for frontend/API links and request URL reconstruction; extend DB migrations and tests accordingly.
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/api_client/services/api_service.go | Record feed events on lifecycle/lint/refresh changes; generate RSS XML for API feed. |
| pkg/api_client/services/api_service_test.go | Extend service tests to assert feed events are created on lifecycle updates and OAS-unreachable refresh. |
| pkg/api_client/services/api_service_oas_test.go | Update repository stub to satisfy extended repository interface. |
| pkg/api_client/routers.go | Register new GET /apis/:id/feed route. |
| pkg/api_client/repositories/api_repositorie.go | Extend repository interface + implement save/list feed event persistence. |
| pkg/api_client/repositories/api_repositorie_test.go | Add DB test for ordering/filtering in ListApiFeedEvents. |
| pkg/api_client/models/impl.go | Remove unused generated ImplResponse type. |
| pkg/api_client/models/api_feed_event.go | Add ApiFeedEvent GORM model + event type constants. |
| pkg/api_client/integration_test.go | Add end-to-end integration test for the feed endpoint response shape. |
| pkg/api_client/helpers/util/url.go | Add helpers for frontend URL and absolute request URL reconstruction. |
| pkg/api_client/helpers/util/url_test.go | Add tests for URL reconstruction behavior and normalization. |
| pkg/api_client/handler/api_handler.go | Add controller handler for RSS feed endpoint. |
| pkg/api_client/handler/api_handler_test.go | Add controller test verifying RSS feed response and self-link generation. |
| pkg/api_client/database/database.go | Add ApiFeedEvent to AutoMigrate list and normalize formatting. |
| api/openapi.json | Document the new /apis/{id}/feed endpoint and RSS response type. |
| .env | Update local env defaults/URLs (Tools endpoint, Typesense flag, endpoints). |
| .changes/unreleased/Added-20260511-230358.yaml | Add changelog entry for the RSS feed feature. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
…revent header spoofing Agent-Logs-Url: https://github.com/developer-overheid-nl/don-api-register/sessions/8959d2cf-0c31-41ce-8251-44a995b0f330 Co-authored-by: pasibun <9033551+pasibun@users.noreply.github.com>
Agent-Logs-Url: https://github.com/developer-overheid-nl/don-api-register/sessions/8959d2cf-0c31-41ce-8251-44a995b0f330 Co-authored-by: pasibun <9033551+pasibun@users.noreply.github.com>
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
…pdate related tests [deploy-test]
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
No description provided.