feat(review): ingest PR-linked issue context

haasonsaas · haasonsaas · commit ac64579ba047 · 2026-03-15T07:53:55.000-07:00
diff --git a/TODO.md b/TODO.md
@@ -78,7 +78,7 @@ This roadmap is derived from deep research into Greptile's public docs, blog, MC
 42. [x] Surface review rule file sources in the Settings UI instead of requiring config edits by hand.
 43. [x] Add structured UI editing for custom context notes, files, and scopes.
 44. [x] Add per-path scoped review instructions in the Settings UI for common repo areas.
-45. [ ] Support Jira/Linear issue context ingestion for PR-linked reviews.
+45. [x] Support Jira/Linear issue context ingestion for PR-linked reviews.
 46. [ ] Support document-backed context ingestion for design docs, RFCs, and runbooks.
 47. [ ] Add explicit "intent mismatch" review checks comparing PR changes to ticket acceptance criteria.
 48. [x] Add review artifacts that show which external context sources influenced a finding.
diff --git a/src/config.rs b/src/config.rs
@@ -116,6 +116,45 @@ pub struct GitHubConfig {
     pub webhook_secret: Option<String>,
 }
 
+#[derive(Debug, Clone, Serialize, Deserialize, Default)]
+pub struct JiraConfig {
+    #[serde(default, rename = "jira_base_url")]
+    pub base_url: Option<String>,
+
+    #[serde(default, rename = "jira_email")]
+    pub email: Option<String>,
+
+    #[serde(default, rename = "jira_api_token")]
+    pub api_token: Option<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize, Default)]
+pub struct LinearConfig {
+    #[serde(default, rename = "linear_api_key")]
+    pub api_key: Option<String>,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq, Hash)]
+#[serde(rename_all = "snake_case")]
+pub enum LinkedIssueProvider {
+    Jira,
+    Linear,
+}
+
+#[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq)]
+pub struct LinkedIssueContext {
+    pub provider: LinkedIssueProvider,
+    pub identifier: String,
+    #[serde(default)]
+    pub title: Option<String>,
+    #[serde(default)]
+    pub status: Option<String>,
+    #[serde(default)]
+    pub url: Option<String>,
+    #[serde(default)]
+    pub summary: String,
+}
+
 #[derive(Debug, Clone, Serialize, Deserialize, Default)]
 pub struct AutomationConfig {
     /// Outbound webhook URL for downstream automation consumers.
@@ -459,6 +498,9 @@ pub struct Config {
     #[serde(default)]
     pub custom_context: Vec<CustomContextConfig>,
 
+    #[serde(default, skip_serializing_if = "Vec::is_empty")]
+    pub linked_issue_contexts: Vec<LinkedIssueContext>,
+
     #[serde(default)]
     pub pattern_repositories: Vec<PatternRepositoryConfig>,
 
@@ -477,6 +519,12 @@ pub struct Config {
     #[serde(default, flatten)]
     pub github: GitHubConfig,
 
+    #[serde(default, flatten)]
+    pub jira: JiraConfig,
+
+    #[serde(default, flatten)]
+    pub linear: LinearConfig,
+
     #[serde(default, flatten)]
     pub automation: AutomationConfig,
 
@@ -679,12 +727,15 @@ impl Default for Config {
             exclude_patterns: default_exclude_patterns(),
             paths: HashMap::new(),
             custom_context: Vec::new(),
+            linked_issue_contexts: Vec::new(),
             pattern_repositories: Vec::new(),
             rules_files: Vec::new(),
             max_active_rules: default_max_active_rules(),
             rule_priority: Vec::new(),
             providers: HashMap::new(),
             github: GitHubConfig::default(),
+            jira: JiraConfig::default(),
+            linear: LinearConfig::default(),
             automation: AutomationConfig::default(),
             server_security: ServerSecurityConfig::default(),
             multi_pass_specialized: false,
diff --git a/src/core/context_provenance.rs b/src/core/context_provenance.rs
@@ -10,6 +10,12 @@ pub enum ContextProvenance {
         name: String,
     },
     CustomContextNotes,
+    JiraIssueContext {
+        issue_key: String,
+    },
+    LinearIssueContext {
+        issue_id: String,
+    },
     DependencyGraphNeighborhood,
     PathSpecificFocusAreas,
     RepositoryGraphMetadata,
@@ -53,6 +59,18 @@ impl ContextProvenance {
         }
     }
 
+    pub fn jira_issue_context(issue_key: impl Into<String>) -> Self {
+        Self::JiraIssueContext {
+            issue_key: issue_key.into(),
+        }
+    }
+
+    pub fn linear_issue_context(issue_id: impl Into<String>) -> Self {
+        Self::LinearIssueContext {
+            issue_id: issue_id.into(),
+        }
+    }
+
     pub fn semantic_retrieval(similarity: f32, symbol_name: impl Into<String>) -> Self {
         Self::SemanticRetrieval {
             similarity,
@@ -100,6 +118,8 @@ impl ContextProvenance {
             }
             Self::Analyzer { .. }
             | Self::CustomContextNotes
+            | Self::JiraIssueContext { .. }
+            | Self::LinearIssueContext { .. }
             | Self::DependencyGraphNeighborhood
             | Self::PathSpecificFocusAreas
             | Self::RepositoryGraphMetadata
@@ -121,6 +141,8 @@ impl ContextProvenance {
         let source = match self {
             Self::ActiveReviewRules | Self::Analyzer { .. } => return None,
             Self::CustomContextNotes => "custom-context".to_string(),
+            Self::JiraIssueContext { .. } => "jira-issue".to_string(),
+            Self::LinearIssueContext { .. } => "linear-issue".to_string(),
             Self::DependencyGraphNeighborhood => "dependency-graph".to_string(),
             Self::PathSpecificFocusAreas => "path-focus".to_string(),
             Self::RepositoryGraphMetadata => "repository-graph".to_string(),
@@ -143,6 +165,10 @@ impl ContextProvenance {
             Self::ActiveReviewRules => "active review rules".to_string(),
             Self::Analyzer { name } => format!("{name} analyzer"),
             Self::CustomContextNotes => "custom context notes".to_string(),
+            Self::JiraIssueContext { issue_key } => format!("jira issue context: {issue_key}"),
+            Self::LinearIssueContext { issue_id } => {
+                format!("linear issue context: {issue_id}")
+            }
             Self::DependencyGraphNeighborhood => "dependency graph neighborhood".to_string(),
             Self::PathSpecificFocusAreas => "path-specific focus areas".to_string(),
             Self::RepositoryGraphMetadata => "repository graph metadata".to_string(),
@@ -234,6 +260,14 @@ mod tests {
             ContextProvenance::RepositoryGraphMetadata.to_string(),
             "repository graph metadata"
         );
+        assert_eq!(
+            ContextProvenance::jira_issue_context("ENG-123").to_string(),
+            "jira issue context: ENG-123"
+        );
+        assert_eq!(
+            ContextProvenance::linear_issue_context("LIN-42").to_string(),
+            "linear issue context: LIN-42"
+        );
     }
 
     #[test]
@@ -256,6 +290,18 @@ mod tests {
                 .as_deref(),
             Some("context-source:symbol-graph")
         );
+        assert_eq!(
+            ContextProvenance::jira_issue_context("ENG-123")
+                .artifact_tag()
+                .as_deref(),
+            Some("context-source:jira-issue")
+        );
+        assert_eq!(
+            ContextProvenance::linear_issue_context("LIN-42")
+                .artifact_tag()
+                .as_deref(),
+            Some("context-source:linear-issue")
+        );
         assert!(ContextProvenance::ActiveReviewRules
             .artifact_tag()
             .is_none());
diff --git a/src/review/context_helpers.rs b/src/review/context_helpers.rs
@@ -5,6 +5,8 @@ mod pattern_repositories;
 #[path = "context_helpers/ranking.rs"]
 mod ranking;
 
-pub use injection::{inject_custom_context, inject_pattern_repository_context};
+pub use injection::{
+    inject_custom_context, inject_linked_issue_context, inject_pattern_repository_context,
+};
 pub use pattern_repositories::{resolve_pattern_repositories, PatternRepositoryMap};
 pub use ranking::rank_and_trim_context_chunks;
diff --git a/src/review/context_helpers/injection.rs b/src/review/context_helpers/injection.rs
@@ -37,6 +37,61 @@ pub async fn inject_custom_context(
     Ok(())
 }
 
+pub fn inject_linked_issue_context(
+    config: &config::Config,
+    diff: &core::UnifiedDiff,
+    context_chunks: &mut Vec<core::LLMContextChunk>,
+) {
+    for issue in &config.linked_issue_contexts {
+        let mut lines = vec![format!(
+            "Linked {} issue context: {}",
+            match issue.provider {
+                config::LinkedIssueProvider::Jira => "Jira",
+                config::LinkedIssueProvider::Linear => "Linear",
+            },
+            issue.identifier
+        )];
+
+        if let Some(title) = issue
+            .title
+            .as_deref()
+            .filter(|value| !value.trim().is_empty())
+        {
+            lines.push(format!("Title: {title}"));
+        }
+        if let Some(status) = issue
+            .status
+            .as_deref()
+            .filter(|value| !value.trim().is_empty())
+        {
+            lines.push(format!("Status: {status}"));
+        }
+        if let Some(url) = issue
+            .url
+            .as_deref()
+            .filter(|value| !value.trim().is_empty())
+        {
+            lines.push(format!("URL: {url}"));
+        }
+        if !issue.summary.trim().is_empty() {
+            lines.push("Summary:".to_string());
+            lines.push(issue.summary.trim().to_string());
+        }
+
+        context_chunks.push(
+            core::LLMContextChunk::documentation(diff.file_path.clone(), lines.join("\n"))
+                .with_provenance(match issue.provider {
+                    config::LinkedIssueProvider::Jira => {
+                        core::ContextProvenance::jira_issue_context(issue.identifier.clone())
+                    }
+                    config::LinkedIssueProvider::Linear => {
+                        core::ContextProvenance::linear_issue_context(issue.identifier.clone())
+                    }
+                }),
+        );
+    }
+}
+
 pub async fn inject_pattern_repository_context(
     config: &config::Config,
     resolved_repositories: &PatternRepositoryMap,
@@ -149,6 +204,50 @@ mod tests {
         );
     }
 
+    #[test]
+    fn inject_linked_issue_context_uses_provider_specific_provenance() {
+        let mut config = config::Config::default();
+        config.linked_issue_contexts = vec![
+            config::LinkedIssueContext {
+                provider: config::LinkedIssueProvider::Jira,
+                identifier: "ENG-123".to_string(),
+                title: Some("Keep API status enum aligned".to_string()),
+                status: Some("In Progress".to_string()),
+                url: Some("https://example.atlassian.net/browse/ENG-123".to_string()),
+                summary: "The API contract must remain backwards compatible.".to_string(),
+            },
+            config::LinkedIssueContext {
+                provider: config::LinkedIssueProvider::Linear,
+                identifier: "OPS-9".to_string(),
+                title: Some("Propagate webhook secret rename".to_string()),
+                status: Some("Todo".to_string()),
+                url: Some(
+                    "https://linear.app/evalops/issue/OPS-9/rename-webhook-secret".to_string(),
+                ),
+                summary: "Deployment manifests should use the new secret name.".to_string(),
+            },
+        ];
+
+        let mut context_chunks = Vec::new();
+        inject_linked_issue_context(&config, &diff_for("src/lib.rs"), &mut context_chunks);
+
+        assert_eq!(context_chunks.len(), 2);
+        assert_eq!(
+            context_chunks[0].provenance,
+            Some(core::ContextProvenance::jira_issue_context("ENG-123"))
+        );
+        assert!(context_chunks[0]
+            .content
+            .contains("Keep API status enum aligned"));
+        assert_eq!(
+            context_chunks[1].provenance,
+            Some(core::ContextProvenance::linear_issue_context("OPS-9"))
+        );
+        assert!(context_chunks[1]
+            .content
+            .contains("Deployment manifests should use the new secret name."));
+    }
+
     #[tokio::test]
     async fn inject_pattern_repository_context_tags_source_and_chunks() {
         let dir = tempfile::tempdir().unwrap();
diff --git a/src/review/pipeline/file_context/sources/repo.rs b/src/review/pipeline/file_context/sources/repo.rs
@@ -9,6 +9,11 @@ pub(in super::super) async fn inject_repository_context(
     diff: &core::UnifiedDiff,
     context_chunks: &mut Vec<core::LLMContextChunk>,
 ) -> Result<()> {
+    super::super::super::super::context_helpers::inject_linked_issue_context(
+        &services.config,
+        diff,
+        context_chunks,
+    );
     super::super::super::super::context_helpers::inject_custom_context(
         &services.config,
         &services.context_fetcher,
diff --git a/src/server/api.rs b/src/server/api.rs
@@ -226,12 +226,14 @@ mod tests {
             "github_webhook_secret".to_string(),
             serde_json::json!("secret5"),
         );
+        obj.insert("jira_api_token".to_string(), serde_json::json!("secret6"));
+        obj.insert("linear_api_key".to_string(), serde_json::json!("secret7"));
         obj.insert(
             "automation_webhook_secret".to_string(),
-            serde_json::json!("secret6"),
+            serde_json::json!("secret8"),
         );
-        obj.insert("server_api_key".to_string(), serde_json::json!("secret7"));
-        obj.insert("vault_token".to_string(), serde_json::json!("secret8"));
+        obj.insert("server_api_key".to_string(), serde_json::json!("secret9"));
+        obj.insert("vault_token".to_string(), serde_json::json!("secret10"));
         mask_config_secrets(&mut obj);
         assert_eq!(obj.get("api_key").unwrap(), &serde_json::json!("***"));
         assert_eq!(obj.get("github_token").unwrap(), &serde_json::json!("***"));
@@ -247,6 +249,14 @@ mod tests {
             obj.get("github_webhook_secret").unwrap(),
             &serde_json::json!("***")
         );
+        assert_eq!(
+            obj.get("jira_api_token").unwrap(),
+            &serde_json::json!("***")
+        );
+        assert_eq!(
+            obj.get("linear_api_key").unwrap(),
+            &serde_json::json!("***")
+        );
         assert_eq!(
             obj.get("automation_webhook_secret").unwrap(),
             &serde_json::json!("***")
diff --git a/src/server/api/admin.rs b/src/server/api/admin.rs
@@ -132,6 +132,8 @@ pub(crate) fn mask_config_secrets(obj: &mut serde_json::Map<String, serde_json::
         "github_client_secret",
         "github_private_key",
         "github_webhook_secret",
+        "jira_api_token",
+        "linear_api_key",
         "automation_webhook_secret",
         "server_api_key",
         "vault_token",
diff --git a/src/server/api/gh.rs b/src/server/api/gh.rs
