Skip to content

Template Manager plugin: build fix, manifest completeness, in-app help & icons - #51

Merged
hal-eisen-adfa merged 4 commits into
mainfrom
ADFA-4857-2-plugins-from-Jim
Jul 25, 2026
Merged

Template Manager plugin: build fix, manifest completeness, in-app help & icons#51
hal-eisen-adfa merged 4 commits into
mainfrom
ADFA-4857-2-plugins-from-Jim

Conversation

@hal-eisen-adfa

Copy link
Copy Markdown
Contributor

Summary

Makes the Template Manager plugin (browse / install / uninstall / delete .cgt project templates on-device) submission-ready. Fixes the build, completes the manifest, adds full in-app help, and gives it real icons. All changes were verified end-to-end on an emulator (Code On The Go ≈ 26.29), not just by a passing build.

Build & manifest

  • Point at the shared repo-root jars (../libs/plugin-api.jar, ../libs/gradle-plugin.jar) instead of a non-existent local libs/ — the plugin now builds from a clean checkout.
  • Add plugin.editor_tabs (the plugin contributes editor tabs), plugin.max_ide_version=26.30 + real min_ide_version=26.29, and filesystem.read (the code reads files). Least-privilege permission comment rewritten.

In-app help (Tier 1/2/3)

  • Add the required top-level templatemanagerplugin.html install-decision page.
  • Add Tier 3 offline docs at assets/docs/index.html, wire getTier3DocsAssetPath() + a PluginTooltipButton, and set tooltipTag on the sidebar item and editor tab.
  • Fix a latent tooltip defect: help rendered "n/a". Root-caused via the on-device documentation.db — the entry registered correctly but the 2-arg showTooltip(view, tag) looked it up under the wrong category. Switched to the 3-arg showTooltip(view, "plugin_<pluginId>", tag) overload. Verified all three tiers now render.

UX & polish

  • Cards are now titled with the .cgt file name (not the first bundled template's name); a multi-template file shows "Contains N templates" instead of a single template's description.
  • New day/night Plugin Manager icons — a .cgt file with a folded amber corner and bold "CGT" lettering (192×192).
  • Display name is "Template Manager" everywhere users see it (code identifiers unchanged); user-facing text uses the full "Code On The Go".

Verification

Installed the built .cgp on emulator-5554: version check accepted, sidebar opens the editor tab, template list scans the store (shows the bundled core file with all 9 templates), tooltip Tier 1/2/3 all render, new icon and "Template Manager" name shown in the Plugin Manager.

- build: reference shared ../libs jars instead of a missing local libs/
- manifest: add plugin.editor_tabs, max_ide_version (26.29–26.30), and
  filesystem.read; rewrite permissions comment for least privilege
- in-app help: add top-level HTML doc + Tier 3 offline docs/index.html,
  wire tooltipTags and the Tier 3 button, and fix the tooltip category so
  the 3-arg showTooltip resolves the entry (was rendering "n/a")
- cards: title with the .cgt file name; hide first-template description on
  multi-template files
- icons: new 192px day/night CGT file glyphs
- naming: display name "Template Manager"; full "Code On The Go" in
  user-facing text

Verified end-to-end on emulator-5554 (Code On The Go ~26.29).

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

…me convention

- CLAUDE.md: add "In-app help wiring (DocumentationExtension)" recipe — the
  deterministic tooltip contract (category = plugin_<pluginId>, 3-arg
  showTooltip, Tier 3 setup, documentation.db debug query) so every plugin
  gets it right without a device round-trip
- plugin-review skill: RUBRIC 6.7 now fails a wrong category / bare 2-arg
  showTooltip (the "n/a" trap); SKILL 6.6 disambiguates the doc filename to
  the kebab folder-name form (template-manager.html)
- learnings: tooltip n/a root-cause + Glide icon-cache invalidation
- retrospective: 2026-07-24 session log
@hal-eisen-adfa
hal-eisen-adfa merged commit a61431e into main Jul 25, 2026
@hal-eisen-adfa
hal-eisen-adfa deleted the ADFA-4857-2-plugins-from-Jim branch July 25, 2026 00:24
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.

1 participant