From c8b848d89bd397ea0c4132b7aeb2417221f7bdd5 Mon Sep 17 00:00:00 2001
From: Rahix <rahix@rahix.de>
Date: Fri, 17 Oct 2025 12:46:14 +0200
Subject: [PATCH] Ignore external issue dependencies

Issues in forgejo can also depend on issues from other repositories.
Ignore such external dependencies when building the techtree.
---
 techtree-manager/src/collect.rs | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/techtree-manager/src/collect.rs b/techtree-manager/src/collect.rs
index c0034ed..b02afca 100644
--- a/techtree-manager/src/collect.rs
+++ b/techtree-manager/src/collect.rs
@@ -68,6 +68,21 @@ pub async fn collect_tree(ctx: &crate::Context) -> anyhow::Result<crate::tree::T
             .with_context(|| format!("Failed to fetch issue dependencies for #{issue}",))?;
 
         for dep in dependencies {
+            // Check that the dependency is actually an issue in the techtree and not external
+            let dep_repo = dep
+                .repository
+                .context("Dependency issue without repository info")?;
+            if dep_repo.owner.as_ref() != Some(&ctx.owner)
+                || dep_repo.name.as_ref() != Some(&ctx.repo)
+            {
+                log::warn!(
+                    "Issue #{issue} depends on external {}#{}, ignoring.",
+                    dep_repo.full_name.as_deref().unwrap_or("unknown?"),
+                    dep.number.unwrap_or(9999)
+                );
+                continue;
+            }
+
             let dep_number = dep.number.context("Missing issue number in dependency")?;
             if !tree.find_element_by_issue_number(dep_number).is_some() {
                 log::warn!("Found dependency from #{issue} on non-tracked issue #{dep_number}!");