Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 20 additions & 21 deletions performance/r8-analyzer/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,15 +16,15 @@ metadata:
- optimization
---

## Step 1. Setup and Configuration Check
## Step 1. Setup and configuration check

- Inspect `build.gradle`, `build.gradle.kts`, and `gradle.properties`.
- Use [/references/CONFIGURATION.md](references/CONFIGURATION.md) to identify missing optimizations.
- Use [references/CONFIGURATION.md](references/CONFIGURATION.md) to identify missing optimizations.
- **AGP** : If \< 9.0, suggest migration to 9.0 for [build time improvement
performance](references/android/topic/performance/app-optimization/enable-app-optimization.md)
- **Full Mode** : Verify `android.enableR8.fullMode=false` is removed from gradle.properties.
- **Full mode** : Verify `android.enableR8.fullMode=false` is removed from gradle.properties.

## Step 2. Analysis Path Selection
## Step 2. Analysis path selection

- Inspect `build.gradle`, `build.gradle.kts`, and `gradle.properties`
and `libs.versions.toml` to
Expand All @@ -34,39 +34,38 @@ metadata:

- **If R8 \< 9.3.7-dev** : Proceed to **Path B (Heuristic)**.

### Path A: Quantitative Data Generation (R8 \>= 9.3.7-dev)
### Path A: Quantitative data generation (R8 \>= 9.3.7-dev)

- **Check Requirements** : Python and `protobuf` package are mandatory.
- **Generate and Analyze** : You MUST run the shell commands described in `[/references/CONFIGURATION-ANALYZER.md][7]` to generate the proto file using R8 configuration analyzer, convert it to json and analyze the result.
- **Check requirements** : Python and `protobuf` package are mandatory.
- **Generate and analyze** : You MUST run the shell commands described in [references/CONFIGURATION-ANALYZER.md](references/CONFIGURATION-ANALYZER.md) to generate the proto file using R8 configuration analyzer, convert it to json and analyze the result.
- **Report**: Rely entirely on the generated file analysis.txt for scores and rule impact metrics. Proceed to Step 3.

### Path B: Heuristic Evaluation and Recommendation (R8 \< 9.3.7-dev)
### Path B: Heuristic evaluation and recommendation (R8 \< 9.3.7-dev)

*(Use ONLY if quantitative data generation is not possible)*

- **Manual Evaluation** : Inspect `proguard-rules.pro`.
- **Library Check** : Compare rules against [/references/REDUNDANT-RULES.md](references/REDUNDANT-RULES.md) .Suggest **Remove** for bundled rules.
- **Custom Rule Check** : Use [/references/KEEP-RULES-IMPACT-HIERARCHY.md](references/KEEP-RULES-IMPACT-HIERARCHY.md) and [/references/REFLECTION-GUIDE.md](references/REFLECTION-GUIDE.md) to prioritize and evaluate. Suggest **Refine** for broad rules (e.g., package-wide).
- **Validation** : Suggest Macrobenchmark tests using [ui automator](references/android/training/testing/other-components/ui-automator.md) for any proposed changes. Proceed to Step 3.
- **Manual evaluation** : Inspect `proguard-rules.pro`.
- **Library check** : Compare rules against [references/REDUNDANT-RULES.md](references/REDUNDANT-RULES.md). Suggest **Remove** for bundled rules.
- **Custom rule check** : Use [references/KEEP-RULES-IMPACT-HIERARCHY.md](references/KEEP-RULES-IMPACT-HIERARCHY.md) and [references/REFLECTION-GUIDE.md](references/REFLECTION-GUIDE.md) to prioritize and evaluate. Suggest **Refine** for broad rules (for example, package-wide).
- **Validation** : Suggest Macrobenchmark tests using [UI Automator](references/android/training/testing/other-components/ui-automator.md) for any proposed changes. Proceed to Step 3.

## Step 3. Report Generation
## Step 3. Report generation

- **Format** : Follow `[/references/REPORT_FORMAT.md][8]` strictly.
- **Format** : Follow [references/REPORT_FORMAT.md](references/REPORT_FORMAT.md) strictly.
- **Input**: Extract metrics (Scores, Impacts, Example Classes)
directly from generated file analysis.txt if using Path A,
or from manual findings if using Path B.

- **Output** :
Output ONLY the raw Markdown report in the chat.
Do NOT output conversational filler (e.g., "Here is your report...").
Do NOT output conversational filler (for example, "Here is your report...").
Do NOT provide recommendations, next steps,
or any other text outside of the sections defined in
`[/references/REPORT_FORMAT.md][8]`
[references/REPORT_FORMAT.md](references/REPORT_FORMAT.md)
Do NOT mention the path used for analysis of the configuration

## Constraints

- **Strict Output Limit**: The final output MUST strictly be the Markdown report and nothing else.
- **No Code Changes**: Research and suggest only; Do not modify files.
- **No Redundancy**: Do not explain R8 benefits or reference skill internal files in the report.
- **Focus**: Omit sections (e.g., Subsumed Rules, Configuration) if no issues or items are found.
- **Strict output limit**: The final output MUST strictly be the Markdown report and nothing else.
- **No code changes**: Research and suggest only; Do not modify files.
- **No redundancy**: Do not explain R8 benefits or reference skill internal files in the report.
- **Focus**: Omit sections (for example, Subsumed Rules, Configuration) if no issues or items are found.
Loading
Loading