fix(android): allow and escape Kotlin keyword as package identifier

.changes/android-kotlin-keyword-as-ident.md

@@ -0,0 +1,5 @@
+---
+"cargo-mobile2": "patch"
+---
+
+On Android, allows using Kotlin keywords as identifiers and escape them in templates.

src/config/app/domain.rs

@@ -3,9 +3,8 @@ use std::error::Error;
 use std::fmt;
 
 static RESERVED_PACKAGE_NAMES: [&str; 2] = ["kotlin", "java"];
-static RESERVED_KEYWORDS: [&str; 63] = [
+static RESERVED_JAVA_KEYWORDS: [&str; 50] = [
     "abstract",
-    "as",
     "assert",
     "boolean",
     "break",
@@ -22,26 +21,20 @@ static RESERVED_KEYWORDS: [&str; 63] = [
     "else",
     "enum",
     "extends",
-    "false",
     "final",
     "finally",
     "float",
     "for",
-    "fun",
     "goto",
     "if",
     "implements",
     "import",
     "instanceof",
-    "in",
     "int",
     "interface",
-    "is",
     "long",
     "native",
     "new",
-    "null",
-    "object",
     "package",
     "private",
     "protected",
@@ -57,15 +50,9 @@ static RESERVED_KEYWORDS: [&str; 63] = [
     "throw",
     "throws",
     "transient",
-    "true",
     "try",
-    "typealias",
-    "typeof",
-    "val",
-    "var",
     "void",
     "volatile",
-    "when",
     "while",
 ];
 
@@ -129,7 +116,7 @@ pub fn check_domain_syntax(domain_name: &str) -> Result<(), DomainError> {
         if label.is_empty() {
             return Err(DomainError::EmptyLabel);
         }
-        if RESERVED_KEYWORDS.contains(&label) {
+        if RESERVED_JAVA_KEYWORDS.contains(&label) {
             return Err(DomainError::ReservedKeyword {
                 keyword: label.to_owned(),
             });

src/config/app/mod.rs

@@ -222,6 +222,21 @@ impl App {
             .join(".")
     }
 
+    pub fn android_identifier_escape_kotlin_keyword(&self) -> String {
+        let identifier = format!("{}.{}", self.reverse_domain(), self.name_snake());
+        identifier
+            .split('.')
+            .map(|s| {
+                if crate::reserved_names::KOTLIN_ONLY_KEYWORDS.contains(&s) {
+                    format!("`{}`", s)
+                } else {
+                    s.to_string()
+                }
+            })
+            .collect::<Vec<_>>()
+            .join(".")
+    }
+
     pub fn manifest_path(&self) -> PathBuf {
         self.root_dir().join("Cargo.toml")
     }

src/reserved_names/mod.rs

@@ -19,6 +19,22 @@ pub static WINDOWS: &[&str] = &[
 
 pub static ARTIFACTS: &[&str] = &["deps", "examples", "build", "incremental"];
 
+pub static KOTLIN_ONLY_KEYWORDS: &[&str] = &[
+    "as",
+    "false",
+    "fun",
+    "in",
+    "is",
+    "null",
+    "object",
+    "true",
+    "typealias",
+    "typeof",
+    "val",
+    "var",
+    "when",
+];
+
 pub fn in_keywords(s: impl AsRef<str>) -> bool {
     KEYWORDS.contains(&s.as_ref())
 }

src/templating/init.rs

@@ -6,6 +6,7 @@ use crate::{
         Bicycle, EscapeFn, HelperDef, JsonMap,
     },
     config::{app, Config},
+    reserved_names::KOTLIN_ONLY_KEYWORDS,
     util::{self, Git},
 };
 use std::collections::HashMap;
@@ -122,6 +123,28 @@ fn reverse_domain_snake_case(
         .map_err(Into::into)
 }
 
+fn escape_kotlin_keyword(
+    helper: &Helper,
+    _: &Handlebars,
+    _: &Context,
+    _: &mut RenderContext,
+    out: &mut dyn Output,
+) -> HelperResult {
+    let escaped_result = get_str(helper)
+        .split('.')
+        .map(|s| {
+            if KOTLIN_ONLY_KEYWORDS.contains(&s) {
+                format!("`{}`", s)
+            } else {
+                s.to_string()
+            }
+        })
+        .collect::<Vec<_>>()
+        .join(".");
+
+    out.write(&escaped_result).map_err(Into::into)
+}
+
 fn app_root(ctx: &Context) -> Result<&str, RenderErrorReason> {
     let app_root = ctx
         .data()
@@ -219,6 +242,7 @@ pub fn init(config: Option<&Config>) -> Bicycle {
                 "reverse-domain-snake-case",
                 Box::new(reverse_domain_snake_case),
             );
+            helpers.insert("escape-kotlin-keyword", Box::new(escape_kotlin_keyword));
             helpers.insert("dot-to-slash", Box::new(dot_to_slash));
             if config.is_some() {
                 // don't mix these up or very bad things will happen to all of us

templates/apps/wry/gen/android/app/src/main/kotlin/{{dot-to-slash (reverse-domain app.domain)}}/{{snake-case app.name}}/MainActivity.kt.hbs

@@ -1,3 +1,3 @@
-package {{reverse-domain app.domain}}.{{snake-case app.name}}
+package {{escape-kotlin-keyword (reverse-domain app.domain)}}.{{escape-kotlin-keyword (snake-case app.name)}}
 
 class MainActivity : WryActivity()