Skip to content

fix: resolve CI deno fmt failures from PR #80 with individually documented re-exports#90

Merged
kayodebristol merged 3 commits intomainfrom
copilot/fix-ci-failures-pr-80
Apr 6, 2026
Merged

fix: resolve CI deno fmt failures from PR #80 with individually documented re-exports#90
kayodebristol merged 3 commits intomainfrom
copilot/fix-ci-failures-pr-80

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 6, 2026

PR #80 failed deno fmt --check because its branch was based on a pre-formatting version of main — all new exports in src/praxis.js used single quotes, which deno fmt rejects (29/30 files flagged).

Changes

src/praxis.js

  • Replaced export * from "./rules/index.js" wildcard with 28 individually JSDoc-documented named re-exports, each with @type annotation:
/**
 * Fires an alert when the diff rate exceeds the configured burst threshold.
 * Re-exported from `@plures/chronos/rules` for convenience.
 * @type {object}
 */
export { burstDetectionRule } from "./rules/alerting.js";

src/praxis.d.ts

  • Replaced bulk grouped multi-line exports with individually JSDoc-annotated single-line re-exports per symbol

src/chronicle.d.ts / src/index.d.ts

  • Split export { currentCause, withCause } into two separately documented re-exports at each re-export site

TypeDoc pipeline

  • Added typedoc.json targeting all 11 public .d.ts entry points
  • Added docs:typedoc npm script + typedoc@0.28.18 dev dependency
  • Added docs CI job in ci.yml using Node.js 24 (aligned with package.json engines: ">=24")
  • Added docs/typedoc/ to .gitignore

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dl.deno.land
    • Triggering command: /usr/bin/curl curl -s REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI linked an issue Apr 6, 2026 that may be closed by this pull request
@kayodebristol kayodebristol marked this pull request as ready for review April 6, 2026 20:46
Copilot AI review requested due to automatic review settings April 6, 2026 20:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Copilot AI and others added 2 commits April 6, 2026 20:56
…-exports

Agent-Logs-Url: https://github.com/plures/chronos/sessions/4116607a-19eb-400b-89ef-481a1ac9e6df

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
…uirement

Agent-Logs-Url: https://github.com/plures/chronos/sessions/4116607a-19eb-400b-89ef-481a1ac9e6df

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI failures on PR #80 fix: resolve CI deno fmt failures from PR #80 with individually documented re-exports Apr 6, 2026
Copilot AI requested a review from kayodebristol April 6, 2026 20:59
Copy link
Copy Markdown
Contributor

@kayodebristol kayodebristol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved: CI green + code review complete.

@kayodebristol kayodebristol merged commit 7425c3c into main Apr 6, 2026
4 checks passed
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.

[ci-feedback] Fix CI failures on PR #80

3 participants