Skip to content

feat: add CEDA app-integration skills (docker, streamlit, surfdrive, etl-pipeline, sram-oidc)#46

Open
oinkspook wants to merge 2 commits into
mainfrom
skills/app-integration-skills
Open

feat: add CEDA app-integration skills (docker, streamlit, surfdrive, etl-pipeline, sram-oidc)#46
oinkspook wants to merge 2 commits into
mainfrom
skills/app-integration-skills

Conversation

@oinkspook

Copy link
Copy Markdown

Pull Request Description

Type of Change

  • New feature

Description of Changes

Voegt vijf Claude-skills toe voor het bouwen en integreren van CEDA-apps. Elk is gegrond in bestaande repos/code, niet in aannames:

  • docker — CEDA-containerisatie: uv-based image, non-root runtime user, gepinde apt-packages, hadolint, healthcheck, docker-compose voor lokaal. (bron: text-analysis/Dockerfile + docs/gitlab.md)
  • streamlit — CEDA Streamlit-app-structuur: src/ + frontend/ paginaboom, beide multipage-patronen (screen_scanner auto-discovery én expliciete st.Page), npuls-logo/branding. (bron: streamlit-template-app + 1cijferho)
  • surfdrive — SurfDrive public-share ingest (download_surfdrive_csv → MinIO), SURFDRIVE_* config. (bron: instroom-etl-ho/src/transport.py)
  • etl-pipeline — de instroom-etl transport→MinIO container-vorm, docker-compose env-wiring, deploy per image-digest. (bron: instroom-etl-ho/wo, instroom-ml)
  • sram-oidc — SRAM-login via de OIDC-proxy (proxy.sram.surf.nl), authlib-flow, auth-opt-in fallback. (bron: text-analysis/src/auth.py)

Related Issues

Vervolg op de skill-PR's #42 (workflow) en #43 (SDP/platform).

Testing Instructions

Dependencies

Geen nieuwe. Verwijst naar bestaande tooling/packages (uv, hadolint, authlib, surfdrive/minio_file interne packages).

Additional Information

surfdrive en minio_file zijn CEDA-interne packages, niet gevendord in de ETL-repos — de skill beschrijft het gebruikscontract, niet de implementatie.

Checklist

  • My code follows the project's coding standards
  • I have updated the documentation accordingly (skills zijn zelf de documentatie)

🤖 Generated with Claude Code

…etl-pipeline, sram-oidc)

Vijf skills voor het bouwen en integreren van CEDA-apps, elk gegrond in bestaande repos/code:

- docker: CEDA containerisatie (uv-image, non-root user, gepinde apt-packages, hadolint, healthcheck) — o.b.v. text-analysis/Dockerfile + docs/gitlab.md
- streamlit: CEDA Streamlit-app-structuur (src/ + frontend/ paginaboom, st.navigation, npuls-logo) — o.b.v. streamlit-template-app + 1cijferho
- surfdrive: SurfDrive public-share ingest (download CSV → MinIO) — o.b.v. instroom-etl-ho/src/transport.py
- etl-pipeline: de instroom-etl transport→MinIO container-vorm — o.b.v. instroom-etl-ho/wo + instroom-ml
- sram-oidc: SRAM-login via OIDC-proxy (authlib flow, auth opt-in) — o.b.v. text-analysis/src/auth.py

NB: enkele [[links]] verwijzen naar skills uit PR #43 (gitlab-ci, sdp-onboard, surf-sdp-helm-flux, sdp-secrets-management); die resolven zodra #43 gemerged is.

Co-Authored-By: Claude <noreply@anthropic.com>
Zelfde conventie-alignment als #42/#43 (o.b.v. Steven's review):
- ## When this applies voor deze kennis-skills (docker, streamlit, surfdrive, etl-pipeline, sram-oidc)
- [[skill]] → /skill-naam
- ## Important-sectie met randvoorwaarden + wat de skill NIET doet
- descriptions waren al werkwoord-first (Containerize/Build/Integrate/Add)

Co-Authored-By: Claude <noreply@anthropic.com>
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