refactor(check): introduce LintMessageKind::TypeCheckOnly and from_flags

Author: jong-kyung

packages/cli/binding/src/check/analysis.rs

@@ -39,41 +39,69 @@ pub(super) struct LintFailure {
 pub(super) enum LintMessageKind {
     LintOnly,
     LintAndTypeCheck,
+    // Used when lint rules are skipped but type-check still runs.
+    #[allow(dead_code)]
+    TypeCheckOnly,
 }
 
 impl LintMessageKind {
     pub(super) fn from_lint_config(lint_config: Option<&serde_json::Value>) -> Self {
-        let type_check_enabled = lint_config
-            .and_then(|config| config.get("options"))
-            .and_then(|options| options.get("typeCheck"))
-            .and_then(serde_json::Value::as_bool)
-            .unwrap_or(false);
+        if lint_config_type_check_enabled(lint_config) {
+            Self::LintAndTypeCheck
+        } else {
+            Self::LintOnly
+        }
+    }
 
-        if type_check_enabled { Self::LintAndTypeCheck } else { Self::LintOnly }
+    // Selects a variant from the `(lint, type-check)` enabled tuple callers have
+    // already resolved, avoiding a second config lookup inside the helper.
+    #[allow(dead_code)]
+    pub(super) fn from_flags(lint_enabled: bool, type_check_enabled: bool) -> Self {
+        match (lint_enabled, type_check_enabled) {
+            (true, true) => Self::LintAndTypeCheck,
+            (true, false) => Self::LintOnly,
+            (false, true) => Self::TypeCheckOnly,
+            (false, false) => {
+                unreachable!(
+                    "from_flags called with (false, false) — caller must ensure lint or type-check runs before selecting a message kind"
+                )
+            }
+        }
     }
 
     pub(super) fn success_label(self) -> &'static str {
         match self {
             Self::LintOnly => "Found no warnings or lint errors",
             Self::LintAndTypeCheck => "Found no warnings, lint errors, or type errors",
+            Self::TypeCheckOnly => "Found no type errors",
         }
     }
 
     pub(super) fn warning_heading(self) -> &'static str {
         match self {
             Self::LintOnly => "Lint warnings found",
             Self::LintAndTypeCheck => "Lint or type warnings found",
+            Self::TypeCheckOnly => "Type warnings found",
         }
     }
 
     pub(super) fn issue_heading(self) -> &'static str {
         match self {
             Self::LintOnly => "Lint issues found",
             Self::LintAndTypeCheck => "Lint or type issues found",
+            Self::TypeCheckOnly => "Type errors found",
         }
     }
 }
 
+pub(super) fn lint_config_type_check_enabled(lint_config: Option<&serde_json::Value>) -> bool {
+    lint_config
+        .and_then(|config| config.get("options"))
+        .and_then(|options| options.get("typeCheck"))
+        .and_then(serde_json::Value::as_bool)
+        .unwrap_or(false)
+}
+
 fn parse_check_summary(line: &str) -> Option<CheckSummary> {
     let rest = line.strip_prefix("Finished in ")?;
     let (duration, rest) = rest.split_once(" on ")?;