Skip to content

docs(thermidor): add ADR for simplified internal folder structure#7917

Merged
fbeaudoincoveo merged 4 commits into
mainfrom
KIT-5773-adr-for-package-structure-and-boundaries
Jul 6, 2026
Merged

docs(thermidor): add ADR for simplified internal folder structure#7917
fbeaudoincoveo merged 4 commits into
mainfrom
KIT-5773-adr-for-package-structure-and-boundaries

Conversation

@fbeaudoincoveo

Copy link
Copy Markdown
Contributor

@changeset-bot

changeset-bot Bot commented Jul 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 9fdfaa6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@svcsnykcoveo

svcsnykcoveo commented Jul 2, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@fbeaudoincoveo fbeaudoincoveo changed the title doc(thermidor): add ADR for simplified internal folder structure docs(thermidor): add ADR for simplified internal folder structure Jul 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new ADR (ADR-006) and two annexes documenting a proposed simplification of Thermidor’s internal src/ folder structure, including rationale, migration steps, and an example ESLint import-boundary enforcement approach.

Changes:

  • Add ADR-006 describing the move to a public/ + internal/ layout with barrel exports and lint-enforced boundaries.
  • Add Annex A with the full proposed directory tree, barrel examples, and an example no-restricted-imports rule + import mapping.
  • Add Annex B cataloging the core/interface/ indirection layers that would be removed/merged under the proposed structure.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
packages/thermidor/docs/internal/adr/ADR-006-simplified-folder-structure.md Introduces the ADR decision, rationale, requirements mapping, and rollout plan for the simplified folder structure.
packages/thermidor/docs/internal/adr/ADR-006-annex-a-proposed-structure.md Provides the full proposed directory tree plus barrel and ESLint rule examples.
packages/thermidor/docs/internal/adr/ADR-006-annex-b-removed-indirection.md Details which indirection patterns/files would be removed or merged, and why.

@pkg-pr-new

pkg-pr-new Bot commented Jul 2, 2026

Copy link
Copy Markdown
@coveo/atomic

npm i https://pkg.pr.new/@coveo/atomic@7917

@coveo/atomic-hosted-page

npm i https://pkg.pr.new/@coveo/atomic-hosted-page@7917

@coveo/atomic-legacy

npm i https://pkg.pr.new/@coveo/atomic-legacy@7917

@coveo/atomic-react

npm i https://pkg.pr.new/@coveo/atomic-react@7917

@coveo/auth

npm i https://pkg.pr.new/@coveo/auth@7917

@coveo/bueno

npm i https://pkg.pr.new/@coveo/bueno@7917

@coveo/create-atomic

npm i https://pkg.pr.new/@coveo/create-atomic@7917

@coveo/create-atomic-component

npm i https://pkg.pr.new/@coveo/create-atomic-component@7917

@coveo/create-atomic-component-project

npm i https://pkg.pr.new/@coveo/create-atomic-component-project@7917

@coveo/create-atomic-result-component

npm i https://pkg.pr.new/@coveo/create-atomic-result-component@7917

@coveo/create-atomic-rollup-plugin

npm i https://pkg.pr.new/@coveo/create-atomic-rollup-plugin@7917

@coveo/headless

npm i https://pkg.pr.new/@coveo/headless@7917

@coveo/headless-react

npm i https://pkg.pr.new/@coveo/headless-react@7917

@coveo/shopify

npm i https://pkg.pr.new/@coveo/shopify@7917

commit: 9fdfaa6

Comment thread packages/thermidor/docs/internal/adr/ADR-006-annex-b-removed-indirection.md Outdated
Comment thread packages/thermidor/docs/internal/adr/ADR-006-annex-b-removed-indirection.md Outdated

@louis-bompart louis-bompart left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Some ideas

  • Should we add README.md or other markdown file where we put index.ts to:
    • Explains what's this folder does
    • List the various file

I am thinking that having a "markdown" tree that follow the architecture could help agent follows and load only what's needed

Co-authored-by: fbeaudoincoveo <23503066+fbeaudoincoveo@users.noreply.github.com>
@fbeaudoincoveo
fbeaudoincoveo added this pull request to the merge queue Jul 6, 2026
Merged via the queue into main with commit 36fffdb Jul 6, 2026
46 checks passed
@fbeaudoincoveo
fbeaudoincoveo deleted the KIT-5773-adr-for-package-structure-and-boundaries branch July 6, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants