feat(atomic-a11y): scaffold new package for accessibility tooling#7111
feat(atomic-a11y): scaffold new package for accessibility tooling#7111
Conversation
Scaffolds the new @coveo/atomic-a11y package for accessibility tooling extraction. Includes package.json with axe-core and vitest dependencies, TypeScript configuration, and workspace registration.
b650973 to
7bd623e
Compare
There was a problem hiding this comment.
Pull request overview
Scaffolds a new workspace package, @coveo/atomic-a11y, to host upcoming accessibility tooling for Atomic (foundation for the linked PR chain).
Changes:
- Added
packages/atomic-a11y/with initialpackage.json,tsconfig.json,turbo.json, and an empty barrel export. - Registered the new package in the pnpm workspace (and updated the lockfile importer section accordingly).
- Updated cspell dictionary entries to accommodate upcoming a11y terminology used in the docs.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Adds packages/atomic-a11y to the workspace package globs. |
| pnpm-lock.yaml | Adds an importer entry for the new package’s devDependencies. |
| packages/atomic-a11y/package.json | Defines the new private package metadata and build script. |
| packages/atomic-a11y/tsconfig.json | Establishes TS build output and NodeNext module settings for the new package. |
| packages/atomic-a11y/turbo.json | Adds package-level Turbo task configuration for build. |
| packages/atomic-a11y/src/index.ts | Empty barrel module placeholder. |
| packages/atomic-a11y/PR-DEPENDENCY-GRAPH.md | Documents the intended PR merge chain for the a11y workstream. |
| .cspell.json | Adds openacr and VPAT to the workspace dictionary. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
| { | ||
| "extends": ["//"], | ||
| "tasks": { | ||
| "build": { |
There was a problem hiding this comment.
packages/atomic-a11y/turbo.json defines a package-level build task without dependsOn: ["^build"]. In this repo, package turbo.json task definitions override the root defaults (e.g., packages/atomic/turbo.json), so this drops the workspace-wide build dependency chain and can lead to incorrect build ordering once atomic-a11y depends on other workspace packages. Add dependsOn: ["^build"] (or remove this override if it’s not needed) to keep build behavior consistent with other packages like packages/atomic-angular/turbo.json.
| "build": { | |
| "build": { | |
| "dependsOn": ["^build"], |
98356de to
6f08cb4
Compare
Scaffolds the new
@coveo/atomic-a11ypackage for accessibility tooling. This is the root PR for the a11y package chain.What's included
packages/atomic-a11y/withpackage.json,tsconfig.json,turbo.jsonpnpm-workspace.yaml, rootpackage.json)PR Chain (1 of 7)
feat/a11y-package-scaffoldfeat/a11y-shared-foundationfeat/a11y-reporterfeat/a11y-openacrfeat/a11y-scriptsfeat/a11y-reporter-wiringfeat/a11y-ci-integrationKIT-5472