Commit 282a608
authored
ADFA-5598 | Addons fixes: product name, AI Code Suggestions rename, AI Core statements (#94)
* ADFA-5598 | Write "Code On The Go" in full in the READMEs
The product name appeared as CodeOnTheGo, CoGo, and several capitalization
variants of "Code on the Go". Normalize every occurrence to "Code On The Go",
the form tools/addons check.py demands on a published gallery page.
Repository URLs, checkout paths (../CodeOnTheGo, .cache/CodeOnTheGo/,
<path-to-CodeOnTheGo>), and the literal GitHub Action name "Update libs from
CodeOnTheGo" keep the old spelling: they name a repository or a workflow, not
the product.
Directories on tools/addons/skip.txt are untouched, so that the pull requests
in flight against the AI addons do not conflict.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Rename Code Suggestions to AI Code Suggestions and skip it
The addon needs AI Core for inference, so the name now says so. The directory
name drives the slug (ai-code-suggestions) and the display name (AI Code
Suggestions); rootProject.name, pluginName, plugin.name, the gallery page
filename, its title and heading, and the README table row all follow.
Add it to tools/addons/skip.txt: AI Core is held back for the pull requests in
flight against it, so the pair must reach the gallery together.
The Java package, applicationId, plugin.id and plugin.main_class do not change.
Every install is keyed on plugin.id, and a new id orphans the installs that
already exist.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Move AI Code Suggestions to its own package, state the AI Core need
Rename the Java package to com.itsaky.androidide.plugins.aicodesuggestions and
follow it through namespace, applicationId, plugin.id and plugin.main_class.
getTooltipCategory() moves with plugin.id: the host derives the same string to
resolve a lookup, so a category that lags behind the id renders "n/a" at
runtime while the build stays green.
This changes plugin.id, so an existing install of the old id does not upgrade
to this one. The addon is on skip.txt and has never reached the gallery, so
there is nothing published to orphan.
State the AI Core requirement at the top of the README and the gallery page.
The plugin holds no model; it asks AI Core for every completion and shows
nothing until AI Core is installed and a model is selected.
Correct the build instructions: the addon carries its own Gradle wrapper, so
the commands are ./gradlew from plugins/AI-Code-Suggestions, and plugin-api.jar
resolves from ../../libs/.
Verified: ./gradlew assemblePlugin succeeds and packages both icons and
assets/docs/index.html. Not verified on a device.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Correct the AI Core statement in the Speech to Text README
The README said "Requires the AI Core plugin." three lines below a blockquote
that said the opposite, and the architecture diagram, the feature list, and
addon.json all call AI Core optional. The code agrees with them: when
LlmInferenceService is absent, handleTranscript falls back to the raw
transcript so speech is never dropped. Delete the false line and lead with what
the plugin does with AI Core and without it.
Correct the build instructions as well: the addon carries its own Gradle
wrapper, so the commands are ./gradlew from plugins/Speech-to-Text, and
plugin-api.jar resolves from ../../libs/.
Documentation only. The name, the slug, plugin.id and the gallery status do not
change, so existing installs still upgrade in place.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Say the AI Core requirement in the AI Code Suggestions summary
summary is the only addon.json text a browsing user reads: the gallery card
renders it and nothing else in prose. Someone who installs from the card
without opening the details page is exactly the person who ends up with a
plugin that appears to do nothing, so the card has to carry the requirement.
64 of the 120 characters the schema allows.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Write the product name as "Code on the Go"
The earlier commits on this branch used "Code On The Go". That form is wrong.
The app itself writes "Code on the Go" -- the About entry, the terminal banner,
and the editor's own help text all use it. Correct every file this branch
touched.
Also rename the strings the plugin shows at runtime. The tooltip summary, the
tooltip detail and the Tier 3 button still said "Code Suggestions" after the
addon was renamed; the on-device Tooltips table showed the stale text.
Verified on emulator-5554 against Code on the Go C-r-0908-1128: the plugin
installs, the Extensions Manager lists it as "AI Code Suggestions", it stays
enabled across a restart, and the documentation.db row now reads
plugin_com.itsaky.androidide.plugins.aicodesuggestions with the corrected
summary -- so the tooltip category tracks the new plugin.id.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Sweep "Code On The Go" out of plugins, the site, and the tooling
The app writes "Code on the Go". Correct every remaining occurrence under
plugins/ -- READMEs, gallery pages, in-app help, addon.json, manifests and
Kotlin sources alike -- plus:
* check.py, whose own message told contributors to write the wrong form, so
the lint taught the error it exists to catch.
* The gallery site's title, meta description and page-template title. Their
body links already said "Code on the Go", so each page contradicted itself.
* The README that tarball.py generates into every published source tarball,
and its test.
* model.py's docstring, the last one left in that package.
Verified: 88 tests pass and addons check is clean.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD
* ADFA-5598 | Write "Code on the Go" in the skill, the records and the scripts
Finish the sweep. The plugin-review skill and its rubric mattered most: they
are the live instructions a review follows, so they would have kept putting
"Code On The Go" into every new plugin.
The design records are corrected at the source of the error too. The addon
distribution plan states the naming rule in rule 6, and it stated the rule in
the wrong form, which is where this branch first took it from.
The rest are comments in .githooks/pre-push and scripts/check-toolchain.sh,
plus two operator messages the toolchain script prints.
Verified: both shell files parse, and the 88 addons tests still pass.
Claude-Session: https://claude.ai/code/session_01WTwoSBqWoqAmb8vSGjodHD1 parent 44aa624 commit 282a608
81 files changed
Lines changed: 255 additions & 240 deletions
File tree
- .claude/skills/plugin-review
- references
- .githooks
- docs
- process
- superpowers
- plans
- specs
- plugins
- AI-Code-Suggestions
- gradle/wrapper
- src/main
- assets
- docs
- kotlin/com/itsaky/androidide/plugins/aicodesuggestions
- res/values
- APK-Analyzer
- Bookshelf
- src/main
- Client-Time-Tracker
- Code-Together
- Favorite-Snippets
- Flutter-Templates
- Get-AI-Models
- Icons-Repository
- Jetpack-Compose-Preview
- Keystore-Generator
- src/main/assets/docs
- Layout-Editor
- Markdown-Previewer
- NDK-Installer
- Project-to-Template
- src/main
- assets/docs
- kotlin/org/appdevforall/projecttotemplate
- Python-Tools
- src/main/kotlin/com/appdevforall/python/plugin
- Rainbow-Brackets
- src/main
- assets/docs
- kotlin/org/appdevforall/rainbowbrackets
- Random-XKCD
- Sketch-to-UI
- Speech-to-Text
- Vector-Search
- Voice-Alerts
- scripts
- site
- tools/addons
- src/addons
- tests
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
100 | 100 | | |
101 | 101 | | |
102 | 102 | | |
103 | | - | |
| 103 | + | |
104 | 104 | | |
105 | 105 | | |
106 | 106 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | | - | |
| 34 | + | |
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
| 14 | + | |
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
| 46 | + | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
| |||
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
68 | | - | |
| 68 | + | |
69 | 69 | | |
70 | 70 | | |
71 | 71 | | |
| |||
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
77 | | - | |
| 77 | + | |
78 | 78 | | |
79 | 79 | | |
80 | 80 | | |
81 | | - | |
| 81 | + | |
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
| |||
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
91 | | - | |
| 91 | + | |
92 | 92 | | |
93 | 93 | | |
94 | 94 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
13 | 13 | | |
14 | | - | |
| 14 | + | |
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| |||
889 | 889 | | |
890 | 890 | | |
891 | 891 | | |
892 | | - | |
| 892 | + | |
893 | 893 | | |
894 | 894 | | |
895 | 895 | | |
| |||
1328 | 1328 | | |
1329 | 1329 | | |
1330 | 1330 | | |
1331 | | - | |
| 1331 | + | |
1332 | 1332 | | |
1333 | 1333 | | |
1334 | 1334 | | |
1335 | | - | |
| 1335 | + | |
1336 | 1336 | | |
1337 | 1337 | | |
1338 | 1338 | | |
| |||
1621 | 1621 | | |
1622 | 1622 | | |
1623 | 1623 | | |
1624 | | - | |
| 1624 | + | |
1625 | 1625 | | |
1626 | 1626 | | |
1627 | 1627 | | |
1628 | | - | |
| 1628 | + | |
1629 | 1629 | | |
1630 | 1630 | | |
1631 | 1631 | | |
| |||
2151 | 2151 | | |
2152 | 2152 | | |
2153 | 2153 | | |
2154 | | - | |
| 2154 | + | |
2155 | 2155 | | |
2156 | 2156 | | |
2157 | 2157 | | |
| |||
0 commit comments