Skip to content

Commit bbe035c

Browse files
authored
Courseware (#832)
PR: - refactors `courses` package into `courseware`, and `Course` main implementation class into `CourseWare`. Better aligned w/ usage - typing information and rendering components for course content - fixes bundling of custom courseware in a standalone app into the JIT studio build for studio mode - **completes** major goals of #791 studio-mode.
2 parents f382e21 + 7417cc9 commit bbe035c

File tree

184 files changed

+366
-650
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

184 files changed

+366
-650
lines changed

.github/workflows/build-platform-ui.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ on:
66
- 'packages/common/**'
77
- 'packages/db/**'
88
- 'packages/common-ui/**'
9-
- 'packages/courses/**'
9+
- 'packages/courseware/**'
1010
- '.github/workflows/build-platform-ui.yml'
1111
jobs:
1212
build-platform-ui-spa:

.github/workflows/courses-test.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
# .github/workflows/courses-test.yml
2-
name: Courses Package Tests
2+
name: CourseWare Package Tests
33

44
on:
55
push:
66
paths:
7-
- 'packages/courses/**'
7+
- 'packages/courseware/**'
88
- '.github/workflows/courses-test.yml'
99
pull_request:
1010
paths:
11-
- 'packages/courses/**'
11+
- 'packages/courseware/**'
1212
- '.github/workflows/courses-test.yml'
1313

1414
jobs:
@@ -42,17 +42,17 @@ jobs:
4242
cd packages/common-ui
4343
yarn build
4444
45-
- name: Build @vue-skuilder/courses
45+
- name: Build @vue-skuilder/courseware
4646
run: |
47-
cd packages/courses
47+
cd packages/courseware
4848
yarn build
4949
50-
- name: Run tests for @vue-skuilder/courses
50+
- name: Run tests for @vue-skuilder/courseware
5151
run: |
52-
cd packages/courses
52+
cd packages/courseware
5353
yarn test
5454
5555
- name: Report test results
5656
if: always()
5757
run: |
58-
echo "Courses package tests completed with status: ${{ job.status }}"
58+
echo "CourseWare package tests completed with status: ${{ job.status }}"

.github/workflows/publish-npm.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ jobs:
4141
VERSION=${{ steps.extract_version.outputs.VERSION }}
4242
4343
# Verify all packages have the correct version
44-
for package in common db common-ui courses client platform-ui standalone-ui express cli; do
44+
for package in common db common-ui courseware client platform-ui standalone-ui express cli; do
4545
PACKAGE_VERSION=$(cd packages/$package && node -p "require('./package.json').version")
4646
if [ "$PACKAGE_VERSION" != "$VERSION" ]; then
4747
echo "❌ Version mismatch in packages/$package: expected $VERSION, found $PACKAGE_VERSION"
@@ -70,8 +70,8 @@ jobs:
7070
echo "Building common-ui package..."
7171
yarn workspace @vue-skuilder/common-ui build
7272
73-
echo "Building courses package..."
74-
yarn workspace @vue-skuilder/courses build
73+
echo "Building courseware package..."
74+
yarn workspace @vue-skuilder/courseware build
7575
7676
echo "Building edit-ui package..."
7777
yarn workspace @vue-skuilder/edit-ui build
@@ -93,7 +93,7 @@ jobs:
9393
9494
- name: Verify build outputs
9595
run: |
96-
for package in common db common-ui courses client platform-ui standalone-ui express cli; do
96+
for package in common db common-ui courseware client platform-ui standalone-ui express cli; do
9797
if [ ! -d "packages/$package/dist" ]; then
9898
echo "Error: dist directory not found for @vue-skuilder/$package"
9999
exit 1
@@ -123,8 +123,8 @@ jobs:
123123
echo "Publishing @vue-skuilder/common-ui..."
124124
cd packages/common-ui && yarn npm publish --access public && cd ../..
125125
126-
echo "Publishing @vue-skuilder/courses..."
127-
cd packages/courses && yarn npm publish --access public && cd ../..
126+
echo "Publishing @vue-skuilder/courseware..."
127+
cd packages/courseware && yarn npm publish --access public && cd ../..
128128
129129
echo "Publishing @vue-skuilder/client..."
130130
cd packages/client && yarn npm publish --access public && cd ../..
@@ -169,7 +169,7 @@ jobs:
169169
echo " - @vue-skuilder/common"
170170
echo " - @vue-skuilder/db"
171171
echo " - @vue-skuilder/common-ui"
172-
echo " - @vue-skuilder/courses"
172+
echo " - @vue-skuilder/courseware"
173173
echo " - @vue-skuilder/client"
174174
echo " - @vue-skuilder/platform-ui"
175175
echo " - @vue-skuilder/standalone-ui"

.github/workflows/standalone-e2e-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ jobs:
4747
yarn workspace @vue-skuilder/common build
4848
yarn workspace @vue-skuilder/db build
4949
yarn workspace @vue-skuilder/common-ui build
50-
yarn workspace @vue-skuilder/courses build
50+
yarn workspace @vue-skuilder/courseware build
5151
5252
- name: Start CouchDB
5353
run: yarn couchdb:start

CLAUDE.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ User often uses dictation software, which, in context, mangles things like `vue`
2525
- Database E2E watch: `yarn workspace @vue-skuilder/e2e-db test:watch` // known broken - only used for bespoke manual testing
2626
- Unit tests (platform-ui): `yarn workspace @vue-skuilder/platform-ui test:unit`
2727
- Unit tests (common-ui): `yarn workspace @vue-skuilder/common-ui test:unit`
28-
- Unit tests (courses): `yarn workspace @vue-skuilder/courses test`
28+
- Unit tests (courses): `yarn workspace @vue-skuilder/courseware test`
2929
- Run single test: `yarn workspace @vue-skuilder/platform-ui test:unit <test-file-path>`
3030
- E2E tests (platform-ui): `yarn workspace @vue-skuilder/platform-ui test:e2e:headless`
3131
- E2E tests (standalone-ui): `yarn workspace @vue-skuilder/standalone-ui test:e2e:headless`
@@ -44,7 +44,7 @@ User often uses dictation software, which, in context, mangles things like `vue`
4444
#### Frontend Packages
4545
- Build platform-ui: `yarn workspace @vue-skuilder/platform-ui build`
4646
- Build common-ui: `yarn workspace @vue-skuilder/common-ui build`
47-
- Build courses: `yarn workspace @vue-skuilder/courses build`
47+
- Build courses: `yarn workspace @vue-skuilder/courseware build`
4848
- Build standalone-ui: `yarn workspace @vue-skuilder/standalone-ui build`
4949

5050
#### CLI & Tools
@@ -61,7 +61,7 @@ User often uses dictation software, which, in context, mangles things like `vue`
6161
- Lint cli: `yarn workspace @vue-skuilder/cli lint:fix`
6262
- Type check express: `yarn workspace @vue-skuilder/express type-check`
6363
- Type check e2e-db: `yarn workspace @vue-skuilder/e2e-db type-check`
64-
- Type check courses: `yarn workspace @vue-skuilder/courses type-check`
64+
- Type check courses: `yarn workspace @vue-skuilder/courseware type-check`
6565

6666
## Style Guidelines
6767
- Use TypeScript with strict typing
@@ -110,7 +110,7 @@ Always resolve to source directories for internal package references:
110110
- `@db``./packages/db/src`
111111
- `@common``./packages/common/src`
112112
- `@cui``./packages/common-ui/src`
113-
- `@courses``./packages/courses/src`
113+
- `@courseware``./packages/courseware/src`
114114
- `@express``./packages/express/src`
115115
- `@pui``./packages/platform-ui/src`
116116
- `@sui``./packages/standalone-ui/src`

0 commit comments

Comments
 (0)