Skip to content

docs: map CellFence package ownership - #37

Merged
aoto-tech merged 7 commits into
aoto-tech:mainfrom
Dibya0912:docs/package-ownership
Sep 7, 2026
Merged

docs: map CellFence package ownership#37
aoto-tech merged 7 commits into
aoto-tech:mainfrom
Dibya0912:docs/package-ownership

Conversation

@Dibya0912

Copy link
Copy Markdown
Contributor

Related Issue

Closes #34

Description

Document which npm package owns each CellFence entry point, including the CLI, engine, schema, plugin API, GitHub Action, MCP proxy, trace utilities, baseline gate, and reporter. This gives new users and contributors a quick package-selection reference.

Validation

  • git diff --check

Generative AI Disclosure

  • Generative AI was used to assist with this code.

Copilot AI lite review requested due to automatic review settings September 6, 2026 14:43

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@aoto-tech
aoto-tech self-requested a review September 6, 2026 14:50

aoto-tech commented Sep 6, 2026

Copy link
Copy Markdown
Owner

@Dibya0912
Thanks for the PR. The direction looks good, but I think it still needs one adjustment to satisfy the acceptance criteria in Issue #34: “Every current workspace package is represented.”

The current table covers the core packages, but please also list the remaining workspace packages explicitly, including:

  • @cellfence/adapter-call-pattern
  • @cellfence/adapter-opentelemetry
  • @cellfence/plugin-agent-budget
  • @cellfence/plugin-blast-radius
  • @cellfence/plugin-dependency-sovereignty
  • @cellfence/plugin-geo-purity
  • @cellfence/plugin-legacy-strangler
  • @cellfence/plugin-quants-trend

The general note about plugin-* and adapter-* packages is still useful, but the package map should make each workspace package visible as its own entry.

@aoto-tech aoto-tech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for the PR. I’m requesting changes for now because this does not yet satisfy the Issue #34 acceptance criterion that every current workspace package is represented.

Please add explicit entries for the remaining current workspace packages, including the adapter and plugin packages, rather than only describing plugin-* and adapter-* generally. Once each workspace package is visible in the package map, this should be ready for another look.

@aoto-tech aoto-tech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@Dibya0912 Thanks for working on this. I still cannot merge it because #34 asks for every current workspace package to be represented. This table covers the main entry points, but it omits the adapter packages and several @cellfence/plugin-* packages. Please add all current packages and link the map from the README or another discoverable docs page, then rerun git diff --check.

@aoto-tech
aoto-tech force-pushed the docs/package-ownership branch from 7df9c4f to 39ca07b Compare September 7, 2026 13:02
@aoto-tech

Copy link
Copy Markdown
Owner

@Dibya0912 I updated this PR to address the requested changes. The package ownership map now lists every current workspace package explicitly, including the adapter packages and all @cellfence/plugin-* packages, and the README now links to the ownership map so it is discoverable. Validation: git diff --check origin/main...HEAD passed locally, and I verified the package list against the workspace package manifests. GitHub Actions completed successfully after rerunning the transient Windows smoke failure.

@aoto-tech aoto-tech left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Follow-up fixes landed and CI is green.

@aoto-tech
aoto-tech merged commit 9cf45a6 into aoto-tech:main Sep 7, 2026
16 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.

Document which npm package owns each CellFence entry point

3 participants