pumuki 6.3.294 → 6.3.295

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [6.3.295] - 2026-05-19
+
+- `PUMUKI-INC-156`: staged PRE_WRITE/PRE_COMMIT no longer blocks an atomic iOS slice with broad file-level SwiftUI skill findings that expose dozens of lines without a precise AST node introduced by the diff. Precise AST/skills findings still block in every severity when they intersect changed lines; broad brownfield file debt is retained as advisory with remediation demanding line/node-level evidence or a dedicated remediation slice.
+
 ## [6.3.283] - 2026-05-19
 
 - `PUMUKI-INC-150`: brownfield XCTest specs that satisfy `makeSUT()` and `trackForMemoryLeaks()` are also exempt from Swift Testing migration assertion findings (`skills.ios.no-xctassert` / `skills.ios.no-xctunwrap`). These rules remain active for modern Swift Testing suites and XCTest specs without an explicit brownfield quality contract.

package/core/facts/detectors/text/ios.test.ts

@@ -31,6 +31,10 @@ import {
   hasSwiftAdHocLoggingUsage,
   hasSwiftAlamofireUsage,
   hasSwiftEmptyCatchUsage,
+  collectSwiftNSErrorThrowLines,
+  hasSwiftNSErrorThrowUsage,
+  collectSwiftPackageBranchDependencyLines,
+  hasSwiftPackageBranchDependencyUsage,
   hasSwiftForEachIndicesUsage,
   hasSwiftForEachSelfIdentityUsage,
   collectSwiftForceCastLines,
@@ -384,6 +388,61 @@ do {
   assert.equal(hasSwiftEmptyCatchUsage(safe), false);
 });
 
+test('hasSwiftNSErrorThrowUsage detecta NSError lanzado directamente y preserva errores tipados', () => {
+  const source = `
+func map(response: HTTPURLResponse) throws {
+  if response.statusCode >= 500 {
+    throw NSError(domain: "Network", code: response.statusCode)
+  }
+}
+`;
+  const safe = `
+enum NetworkError: Error {
+  case server(statusCode: Int)
+}
+
+func map(response: HTTPURLResponse) throws {
+  if response.statusCode >= 500 {
+    throw NetworkError.server(statusCode: response.statusCode)
+  }
+}
+
+let sample = "throw NSError(domain: code:)"
+// throw NSError(domain: "Sample", code: 1)
+`;
+
+  assert.equal(hasSwiftNSErrorThrowUsage(source), true);
+  assert.deepEqual(collectSwiftNSErrorThrowLines(source), [4]);
+  assert.equal(hasSwiftNSErrorThrowUsage(safe), false);
+  assert.deepEqual(collectSwiftNSErrorThrowLines(safe), []);
+});
+
+test('hasSwiftPackageBranchDependencyUsage detecta dependencias SwiftPM por branch y preserva versiones', () => {
+  const source = `
+let package = Package(
+  dependencies: [
+    .package(url: "https://example.com/ui-kit.git", branch: "main"),
+  ]
+)
+`;
+  const safe = `
+let package = Package(
+  dependencies: [
+    .package(url: "https://example.com/ui-kit.git", exact: "1.2.3"),
+    .package(url: "https://example.com/domain.git", from: "2.0.0"),
+  ]
+)
+
+let sample = ".package(url: branch:)"
+// .package(url: "https://example.com/debug.git", branch: "develop")
+`;
+
+  assert.equal(hasSwiftPackageBranchDependencyUsage(source), true);
+  assert.deepEqual(collectSwiftPackageBranchDependencyLines(source), [4]);
+  assert.equal(hasSwiftPackageBranchDependencyUsage(safe), false);
+  assert.deepEqual(collectSwiftPackageBranchDependencyLines(safe), []);
+});
+
 test('hasSwiftNonLazyScrollForEachUsage detecta ScrollView con stack no lazy y preserva LazyVStack', () => {
   const source = `
 struct FeedView: View {

package/core/facts/detectors/text/ios.ts

@@ -698,6 +698,31 @@ export const hasSwiftEmptyCatchUsage = (source: string): boolean => {
   return /\bcatch(?:\s+(?:let|var)\s+[A-Za-z_][A-Za-z0-9_]*)?\s*\{\s*\}/.test(sanitized);
 };
 
+export const collectSwiftNSErrorThrowLines = (source: string): readonly number[] => {
+  return collectSwiftRegexLines(source, /\bthrow\s+NSError\s*\(/g);
+};
+
+export const hasSwiftNSErrorThrowUsage = (source: string): boolean => {
+  return collectSwiftNSErrorThrowLines(source).length > 0;
+};
+
+export const collectSwiftPackageBranchDependencyLines = (source: string): readonly number[] => {
+  const matches: number[] = [];
+  source.split(/\r?\n/).forEach((line, index) => {
+    const sanitized = line
+      .replace(/"(?:\\.|[^"\\])*"/g, '""')
+      .replace(/\/\/.*$/, '');
+    if (/\.\s*package\s*\([^)]*\bbranch\s*:/.test(sanitized)) {
+      matches.push(index + 1);
+    }
+  });
+  return matches;
+};
+
+export const hasSwiftPackageBranchDependencyUsage = (source: string): boolean => {
+  return collectSwiftPackageBranchDependencyLines(source).length > 0;
+};
+
 export const hasSwiftOnAppearTaskUsage = (source: string): boolean => {
   return collectSwiftOnAppearTaskLines(source).length > 0;
 };