Skip to content

ADFA-6142 | Migrate AI Literacy Course into the plugin gallery - #102

Merged
hal-eisen-adfa merged 1 commit into
mainfrom
chore/ADFA-6142-move-ai-literacy-course-into-plugins
Sep 18, 2026
Merged

hal-eisen-adfa merged 1 commit into
mainfrom
chore/ADFA-6142-move-ai-literacy-course-into-plugins

Conversation

@hal-eisen-adfa

@hal-eisen-adfa hal-eisen-adfa commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Moves the AI Literacy Course addon from the repository root into plugins/, so it publishes through the normal addon pipeline instead of staying held back.

Closes ADFA-6142.

Why the directory is AI-Literacy-Course

The catalog requires each hyphen-separated part of a directory name to start with a capital (model.directory_is_valid), so ai-literacy-course cannot be published as-is.

The capitalised name changes nothing else. The slug is the directory lowercased, so it stays ai-literacy-course, and the display name is the directory with spaces, so it stays AI Literacy Course. That means pluginName, plugin.id, plugin.name, rootProject.name and the documentation filename are all untouched.

Changes

Change Where
Move, with history preserved on all 26 files ai-literacy-course/ → plugins/AI-Literacy-Course/
../libs/*.jar → ../../libs/*.jar for the extra directory level build.gradle.kts:60, settings.gradle.kts:15,16
Drop the local Gradle wrapper, use the shared root wrapper gradlew, gradlew.bat, gradle/wrapper/
Add addon.json, which the catalog requires plugins/AI-Literacy-Course/addon.json
Set the page title to exactly AI Literacy Course ai-literacy-course.html:6
Remove the skip entry, correct the header note to "five remaining" tools/addons/skip.txt
Add the Examples row README.md
Correct the heading, build commands and directory tree plugins/AI-Literacy-Course/README.md

Why the skip entry goes

skip.txt held the addon back because "other pull requests are in flight against them". No open pull request touches ai-literacy-course/ — checked #101, #100, #99, #96, #88, #30 and #22 — so the reason no longer holds. The other five AI addons stay skipped and are not touched.

Reviewers: one behaviour change to confirm

update-libs.yml:100 releases plugins/*/build/plugin/*.cgp. At the repository root the course .cgp did not match that glob; under plugins/ it does, so every "Update libs" release will now carry a ~118 MB asset. This is intended — it is a normal plugin now — but it is a real change in release size and worth a conscious ack.

Verification

Check Result
addons check passes
addons discover lists plugins/AI-Literacy-Course as publishable, not skipped
../../gradlew assemblePlugin from the new location BUILD SUCCESSFUL, proving the ../../libs classpath and the shared wrapper both resolve
unzip -l on the .cgp 118 MB, and contains assets/ai-literacy-course.zip, assets/pdfjs.zip, both icons, assets/docs/index.html and the manifest

Not verified on a device. This addon extracts its bundle on-device through CourseInstaller, so a green build is necessary but not sufficient. Nothing in this pull request changes extraction or generation logic, so INSTALL_VERSION is deliberately not bumped.

Note for anyone rebuilding by hand: run downloadAssets as its own invocation before assemblePlugin. Combining them in one Gradle invocation fails the implicit-dependency check, which is pre-existing and not introduced here.

Move the addon from the repository root into `plugins/` so it publishes
through the normal addon pipeline.

The directory is `AI-Literacy-Course`, not `ai-literacy-course`: the
catalog requires each hyphen-separated part to start with a capital.
The derived values do not change — the slug stays `ai-literacy-course`,
so `pluginName`, the documentation filename and `plugin.id` are all
untouched, and the display name stays `AI Literacy Course`.

- `../libs/*.jar` becomes `../../libs/*.jar` in `build.gradle.kts` and
  `settings.gradle.kts`, for the extra directory level.
- Drop the local Gradle wrapper and use the shared root wrapper.
- Add `addon.json`, which the catalog requires.
- Set the documentation page title to `AI Literacy Course` exactly.
- Remove the `skip.txt` entry. No pull request is open against this
  addon, so the reason the entry gives no longer holds.
- Add the README row.

Verified: `addons check` passes, `addons discover` lists the addon as
publishable, and `../../gradlew assemblePlugin` builds a 118 MB `.cgp`
that contains the course ZIP, `pdfjs.zip`, both icons and the Tier 3
documentation. Not verified on a device.

@claude claude Bot 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.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@hal-eisen-adfa
hal-eisen-adfa requested a review from a team September 18, 2026 03:20
@hal-eisen-adfa
hal-eisen-adfa merged commit 9d631b0 into main Sep 18, 2026
1 check passed
@hal-eisen-adfa
hal-eisen-adfa deleted the chore/ADFA-6142-move-ai-literacy-course-into-plugins branch September 18, 2026 15:33
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.

2 participants