MrSphay/intelligence-terminal
1
0
Fork 0
Code Issues 10 Pull Requests Actions Packages Projects Releases Wiki Activity

ACLED credentialed integration needs regression test and diagnostics #3

New Issue
Closed
opened 2026-05-17 11:21:49 +00:00 by MrSphay · 4 comments
MrSphay commented 2026-05-17 11:21:49 +00:00
Owner
Copy Link

Synced from upstream Crucix #33: https://github.com/calesthio/Crucix/issues/33

Current status in this fork: missing ACLED credentials degrade transparently and docs describe OAuth/cookie fallback. What is still missing is a credentialed regression test path and sharper diagnostics for auth-denied vs no-events vs API-shape changes.

Acceptance criteria:

  • Verify ACLED_USER/ACLED_PASSWORD and any alternate credential names against the current ACLED flow.
  • Add a mocked regression test for token success, auth denied, token endpoint failure, and empty-events response.
  • Health/source output must distinguish missing credentials, auth failed, API failed, and valid empty result.
  • Ensure logs never print credentials or bearer tokens.
Synced from upstream Crucix #33: https://github.com/calesthio/Crucix/issues/33 Current status in this fork: missing ACLED credentials degrade transparently and docs describe OAuth/cookie fallback. What is still missing is a credentialed regression test path and sharper diagnostics for auth-denied vs no-events vs API-shape changes. Acceptance criteria: - Verify ACLED_USER/ACLED_PASSWORD and any alternate credential names against the current ACLED flow. - Add a mocked regression test for token success, auth denied, token endpoint failure, and empty-events response. - Health/source output must distinguish missing credentials, auth failed, API failed, and valid empty result. - Ensure logs never print credentials or bearer tokens.
MrSphay commented 2026-05-17 11:58:39 +00:00
Author
Owner
Copy Link

In Bearbeitung durch Codex auf Branch codex/issue-3-acled-diagnostics. Scope: ACLED Credential-Diagnostik schaerfen und gemockte Regressionstests fuer Token-Erfolg, Auth-Denied, Token-Endpoint-Fehler und leere Events. Keine Dashboard/Docker/README-weiten Aenderungen, damit parallele Branches sauber mergbar bleiben.

In Bearbeitung durch Codex auf Branch codex/issue-3-acled-diagnostics. Scope: ACLED Credential-Diagnostik schaerfen und gemockte Regressionstests fuer Token-Erfolg, Auth-Denied, Token-Endpoint-Fehler und leere Events. Keine Dashboard/Docker/README-weiten Aenderungen, damit parallele Branches sauber mergbar bleiben.
MrSphay commented 2026-05-17 12:01:13 +00:00
Author
Owner
Copy Link

In Bearbeitung durch Codex auf Branch codex/issue-3-acled-diagnostics. Scope: ACLED Credential-/Auth-Diagnostik und gemockte Regressionstests. Ich halte die �nderungen auf ACLED-Source, Source-Doku und bestehende Tests begrenzt, damit PR #9/#12 separat mergbar bleiben.

In Bearbeitung durch Codex auf Branch codex/issue-3-acled-diagnostics. Scope: ACLED Credential-/Auth-Diagnostik und gemockte Regressionstests. Ich halte die �nderungen auf ACLED-Source, Source-Doku und bestehende Tests begrenzt, damit PR #9/#12 separat mergbar bleiben.
MrSphay commented 2026-05-17 12:04:06 +00:00
Author
Owner
Copy Link

PR geoeffnet: #13\n\nStatus: Umsetzung fuer Issue #3 ist fertig auf Branch codex/issue-3-acled-diagnostics. ACLED unterscheidet jetzt no_credentials/auth_failed/api_failed/live-empty, redigiert Token/Cookies in Debug-Ausgaben, akzeptiert ACLED_USER/ACLED_USERNAME und hat gemockte Regressionstests. Lokale npm/node Tests nicht ausgefuehrt; Runner soll npm run test:unit pruefen.

PR geoeffnet: https://git.wilkensxl.de/MrSphay/intelligence-terminal/pulls/13\n\nStatus: Umsetzung fuer Issue #3 ist fertig auf Branch codex/issue-3-acled-diagnostics. ACLED unterscheidet jetzt no_credentials/auth_failed/api_failed/live-empty, redigiert Token/Cookies in Debug-Ausgaben, akzeptiert ACLED_USER/ACLED_USERNAME und hat gemockte Regressionstests. Lokale npm/node Tests nicht ausgefuehrt; Runner soll npm run test:unit pruefen.
MrSphay commented 2026-05-17 12:06:29 +00:00
Author
Owner
Copy Link

Achtung Koordination: Beim Push war bereits ein Remote-Branch codex/issue-3-acled-diagnostics vorhanden (5b176851...). Ich habe diesen Branch nicht �berschrieben.

Meine ACLED-Variante liegt auf: codex/issue-3-acled-diagnostics-auth-tests
Commit: b13979
PR: #14

Status: lokale Checks gr�n. PR #14 ist als erg�nzender/alternativer Fix zu pr�fen und sollte mit dem anderen Issue-#3-Branch koordiniert werden.

Achtung Koordination: Beim Push war bereits ein Remote-Branch codex/issue-3-acled-diagnostics vorhanden (5b176851...). Ich habe diesen Branch nicht �berschrieben. Meine ACLED-Variante liegt auf: codex/issue-3-acled-diagnostics-auth-tests Commit: b13979 PR: https://git.wilkensxl.de/MrSphay/intelligence-terminal/pulls/14 Status: lokale Checks gr�n. PR #14 ist als erg�nzender/alternativer Fix zu pr�fen und sollte mit dem anderen Issue-#3-Branch koordiniert werden.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
design-revert
Reverts visual/design changes
 enhancement
New feature or product improvement
 frontend-redesign
Frontend redesign and app-shell/UI work
 marketing-finalization
Product presentation, README, packaging metadata, and customer-facing communication.
 product
User-facing product capability or workflow
 research
Research-originated product idea ready for implementation design
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
MrSphay
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MrSphay/intelligence-terminal#3
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?