@dependabit/github-client 0.1.14 → 0.1.16

package/src/commits.ts

@@ -1,184 +0,0 @@
-/**
- * Commit Analysis
- * Fetch and analyze commits from GitHub API
- */
-
-import type { GitHubClient } from './client.js';
-
-export interface CommitInfo {
-  sha: string;
-  message: string;
-  author: {
-    name: string;
-    email?: string;
-    date: string;
-  };
-  url?: string;
-}
-
-export interface CommitFile {
-  filename: string;
-  status: 'added' | 'removed' | 'modified' | 'renamed' | 'copied' | 'changed' | 'unchanged';
-  additions?: number;
-  deletions?: number;
-  changes?: number;
-  patch?: string;
-}
-
-export interface CommitDiff {
-  sha: string;
-  files: CommitFile[];
-}
-
-export interface ParsedFiles {
-  added: string[];
-  modified: string[];
-  removed: string[];
-}
-
-export interface FetchCommitsOptions {
-  since?: string;
-  until?: string;
-  sha?: string;
-  path?: string;
-  per_page?: number;
-  page?: number;
-}
-
-/**
- * Fetch commits from GitHub API
- */
-export async function fetchCommits(
-  client: GitHubClient,
-  owner: string,
-  repo: string,
-  options: FetchCommitsOptions = {}
-): Promise<CommitInfo[]> {
-  const octokit = client.getOctokit();
-
-  const response = await octokit.rest.repos.listCommits({
-    owner,
-    repo,
-    ...options
-  });
-
-  return response.data.map((commit) => {
-    const info: CommitInfo = {
-      sha: commit.sha,
-      message: commit.commit.message,
-      author: {
-        name: commit.commit.author?.name || 'Unknown',
-        date: commit.commit.author?.date || new Date().toISOString()
-      }
-    };
-
-    if (commit.commit.author?.email) {
-      info.author.email = commit.commit.author.email;
-    }
-
-    if (commit.html_url) {
-      info.url = commit.html_url;
-    }
-
-    return info;
-  });
-}
-
-/**
- * Get detailed diff for a specific commit
- */
-export async function getCommitDiff(
-  client: GitHubClient,
-  owner: string,
-  repo: string,
-  sha: string
-): Promise<CommitDiff> {
-  const octokit = client.getOctokit();
-
-  const response = await octokit.rest.repos.getCommit({
-    owner,
-    repo,
-    ref: sha
-  });
-
-  return {
-    sha: response.data.sha,
-    files: (response.data.files || []).map((file) => {
-      const commitFile: CommitFile = {
-        filename: file.filename,
-        status: file.status as CommitFile['status']
-      };
-
-      if (file.additions !== undefined) commitFile.additions = file.additions;
-      if (file.deletions !== undefined) commitFile.deletions = file.deletions;
-      if (file.changes !== undefined) commitFile.changes = file.changes;
-      if (file.patch !== undefined) commitFile.patch = file.patch;
-
-      return commitFile;
-    })
-  };
-}
-
-/**
- * Parse commit files into categorized lists
- */
-export function parseCommitFiles(files: CommitFile[]): ParsedFiles {
-  const result: ParsedFiles = {
-    added: [],
-    modified: [],
-    removed: []
-  };
-
-  for (const file of files) {
-    if (file.status === 'added') {
-      result.added.push(file.filename);
-    } else if (file.status === 'modified' || file.status === 'changed') {
-      result.modified.push(file.filename);
-    } else if (file.status === 'removed') {
-      result.removed.push(file.filename);
-    }
-  }
-
-  return result;
-}
-
-/**
- * Get commits between two refs
- */
-export async function getCommitsBetween(
-  client: GitHubClient,
-  owner: string,
-  repo: string,
-  base: string,
-  head: string
-): Promise<CommitInfo[]> {
-  const octokit = client.getOctokit();
-
-  const response = await octokit.rest.repos.compareCommits({
-    owner,
-    repo,
-    base,
-    head
-  });
-
-  return response.data.commits.map((commit) => {
-    const info: CommitInfo = {
-      sha: commit.sha,
-      message: commit.commit.message,
-      author: {
-        name: commit.commit.author?.name || 'Unknown',
-        date: commit.commit.author?.date || new Date().toISOString()
-      }
-    };
-
-    if (commit.commit.author?.email) {
-      info.author.email = commit.commit.author.email;
-    }
-
-    if (commit.html_url) {
-      info.url = commit.html_url;
-    }
-
-    return info;
-  });
-}

package/src/feedback.ts

@@ -1,166 +0,0 @@
-/**
- * False positive feedback listener for dependency tracking
- * Monitors GitHub issue labels to collect user feedback on detections
- */
-
-export interface IssueWithLabels {
-  number: number;
-  title: string;
-  labels: Array<string | { name: string }>;
-  created_at?: string;
-}
-
-export interface IssueManagerInterface {
-  listIssues(): Promise<Array<IssueWithLabels>>;
-  getIssue(issueNumber: number): Promise<IssueWithLabels>;
-}
-
-export interface FeedbackConfig {
-  truePositiveLabel?: string;
-  falsePositiveLabel?: string;
-}
-
-export interface FeedbackData {
-  truePositives: Array<{ number: number; title: string; created_at?: string | undefined }>;
-  falsePositives: Array<{ number: number; title: string; created_at?: string | undefined }>;
-  total: number;
-}
-
-export interface FeedbackRate {
-  falsePositiveRate: number;
-  truePositiveRate: number;
-  totalFeedback: number;
-}
-
-export interface CollectOptions {
-  startDate?: Date;
-  endDate?: Date;
-  repository?: string;
-}
-
-/**
- * Listener that monitors issue labels for false positive feedback
- */
-export class FeedbackListener {
-  private issueManager: IssueManagerInterface;
-  private truePositiveLabel: string;
-  private falsePositiveLabel: string;
-
-  constructor(issueManager: IssueManagerInterface, config: FeedbackConfig = {}) {
-    this.issueManager = issueManager;
-    this.truePositiveLabel = config.truePositiveLabel || 'true-positive';
-    this.falsePositiveLabel = config.falsePositiveLabel || 'false-positive';
-  }
-
-  /**
-   * Collect feedback from issues with feedback labels
-   */
-  async collectFeedback(options: CollectOptions = {}): Promise<FeedbackData> {
-    const issues = await this.issueManager.listIssues();
-
-    const truePositives: FeedbackData['truePositives'] = [];
-    const falsePositives: FeedbackData['falsePositives'] = [];
-
-    for (const issue of issues) {
-      // Filter by date range if specified
-      if (options.startDate && issue.created_at) {
-        const issueDate = new Date(issue.created_at);
-        if (issueDate < options.startDate) continue;
-      }
-      if (options.endDate && issue.created_at) {
-        const issueDate = new Date(issue.created_at);
-        if (issueDate > options.endDate) continue;
-      }
-
-      // Filter by repository if specified
-      if (options.repository && (issue as any).repository !== options.repository) {
-        continue;
-      }
-
-      const labels = issue.labels || [];
-      const labelNames = labels.map((l: string | { name: string }) =>
-        typeof l === 'string' ? l : l.name
-      );
-
-      const hasTrue = labelNames.includes(this.truePositiveLabel);
-      const hasFalse = labelNames.includes(this.falsePositiveLabel);
-
-      // Handle issues with both labels as a special case (log warning but count as true positive)
-      if (hasTrue && hasFalse) {
-        console.warn(
-          `Issue #${issue.number} has both true-positive and false-positive labels. Counting as true-positive.`
-        );
-        truePositives.push({
-          number: issue.number,
-          title: issue.title,
-          created_at: issue.created_at
-        });
-      } else if (hasTrue) {
-        truePositives.push({
-          number: issue.number,
-          title: issue.title,
-          created_at: issue.created_at
-        });
-      } else if (hasFalse) {
-        falsePositives.push({
-          number: issue.number,
-          title: issue.title,
-          created_at: issue.created_at
-        });
-      }
-    }
-
-    return {
-      truePositives,
-      falsePositives,
-      total: truePositives.length + falsePositives.length
-    };
-  }
-
-  /**
-   * Calculate false positive rate from collected feedback
-   */
-  async getFeedbackRate(options: CollectOptions = {}): Promise<FeedbackRate> {
-    const feedback = await this.collectFeedback(options);
-
-    if (feedback.total === 0) {
-      return {
-        falsePositiveRate: 0,
-        truePositiveRate: 0,
-        totalFeedback: 0
-      };
-    }
-
-    return {
-      falsePositiveRate: feedback.falsePositives.length / feedback.total,
-      truePositiveRate: feedback.truePositives.length / feedback.total,
-      totalFeedback: feedback.total
-    };
-  }
-
-  /**
-   * Get feedback from recent time window (e.g., last 30 days)
-   */
-  async getRecentFeedback(days: number, referenceDate?: Date): Promise<FeedbackData> {
-    const endDate = referenceDate || new Date();
-    const startDate = new Date(endDate);
-    startDate.setDate(startDate.getDate() - days);
-
-    return this.collectFeedback({ startDate, endDate });
-  }
-
-  /**
-   * Check if a specific issue has feedback label
-   */
-  async monitorIssue(issueNumber: number): Promise<boolean> {
-    const issue = await this.issueManager.getIssue(issueNumber);
-    const labels = issue.labels || [];
-    const labelNames = labels.map((l: string | { name: string }) =>
-      typeof l === 'string' ? l : l.name
-    );
-
-    return (
-      labelNames.includes(this.truePositiveLabel) || labelNames.includes(this.falsePositiveLabel)
-    );
-  }
-}

package/src/index.ts

@@ -1,15 +0,0 @@
-/**
- * @dependabit/github-client - GitHub API client wrapper
- */
-
-export * from './client.js';
-export * from './commits.js';
-export { IssueManager } from './issues.js';
-export type { IssueData, IssueResult, UpdateIssueData } from './issues.js';
-export { ReleaseManager } from './releases.js';
-export type { Release, ReleaseComparison } from './releases.js';
-export { RateLimitHandler } from './rate-limit.js';
-export type { RateLimitInfo, RateLimitStatus, BudgetReservation } from './rate-limit.js';
-export * from './auth.js';
-export { FeedbackListener } from './feedback.js';
-export type { FeedbackConfig, FeedbackData, FeedbackRate } from './feedback.js';

package/src/issues.ts

@@ -1,185 +0,0 @@
-/**
- * Issue Manager
- * Handles GitHub issue creation and management for dependency changes
- */
-
-import { Octokit } from 'octokit';
-
-export interface IssueData {
-  owner: string;
-  repo: string;
-  title: string;
-  body: string;
-  severity: 'breaking' | 'major' | 'minor';
-  dependency: {
-    id: string;
-    url: string;
-  };
-  assignee?: string;
-}
-
-export interface IssueResult {
-  number: number;
-  url: string;
-  labels: string[];
-  assignees?: string[] | undefined;
-}
-
-export interface UpdateIssueData {
-  owner: string;
-  repo: string;
-  issueNumber: number;
-  body: string;
-  severity?: 'breaking' | 'major' | 'minor';
-  append?: boolean;
-}
-
-export class IssueManager {
-  private octokit: Octokit;
-
-  constructor(auth?: string) {
-    this.octokit = new Octokit({
-      auth: auth || process.env['GITHUB_TOKEN']
-    });
-  }
-
-  /**
-   * Creates a new issue for a dependency change
-   */
-  async createIssue(data: IssueData): Promise<IssueResult> {
-    const { owner, repo, title, body, severity, dependency, assignee } = data;
-
-    // Prepare labels
-    const labels = ['dependabit', `severity:${severity}`, 'dependency-update'];
-
-    // Create the issue
-    const response = await this.octokit.rest.issues.create({
-      owner,
-      repo,
-      title,
-      body: this.formatIssueBody(body, dependency),
-      labels,
-      ...(assignee && { assignees: [assignee] })
-    });
-
-    return {
-      number: response.data.number,
-      url: response.data.html_url,
-      labels,
-      ...(assignee && { assignees: [assignee] })
-    };
-  }
-
-  /**
-   * Finds an existing issue for a dependency
-   */
-  async findExistingIssue(params: {
-    owner: string;
-    repo: string;
-    dependencyId: string;
-  }): Promise<IssueResult | null> {
-    const { owner, repo, dependencyId } = params;
-
-    try {
-      // Search for open issues with dependabit label and dependency ID
-      const query = `repo:${owner}/${repo} is:issue is:open label:dependabit ${dependencyId}`;
-
-      const response = await this.octokit.rest.search.issuesAndPullRequests({
-        q: query,
-        per_page: 1
-      });
-
-      if (response.data.items.length === 0) {
-        return null;
-      }
-
-      const issue = response.data.items[0];
-      if (!issue) {
-        return null;
-      }
-
-      return {
-        number: issue.number,
-        url: issue.html_url,
-        labels: issue.labels.map((l) => (typeof l === 'string' ? l : l.name || ''))
-      };
-    } catch (error) {
-      console.error('Error finding existing issue:', error);
-      return null;
-    }
-  }
-
-  /**
-   * Updates an existing issue
-   */
-  async updateIssue(data: UpdateIssueData): Promise<IssueResult> {
-    const { owner, repo, issueNumber, body, severity, append } = data;
-
-    let finalBody = body;
-
-    // If appending, fetch current body first
-    if (append) {
-      const current = await this.octokit.rest.issues.get({
-        owner,
-        repo,
-        issue_number: issueNumber
-      });
-      finalBody = `${current.data.body}\n\n---\n\n${body}`;
-    }
-
-    // Update labels if severity changed
-    const updateParams: {
-      owner: string;
-      repo: string;
-      issue_number: number;
-      body: string;
-      labels?: string[];
-    } = {
-      owner,
-      repo,
-      issue_number: issueNumber,
-      body: finalBody
-    };
-
-    if (severity) {
-      const current = await this.octokit.rest.issues.get({
-        owner,
-        repo,
-        issue_number: issueNumber
-      });
-
-      const existingLabels = current.data.labels
-        .map((l) => (typeof l === 'string' ? l : l.name))
-        .filter((l): l is string => !!l);
-
-      const severityLabels = ['dependabit', `severity:${severity}`, 'dependency-update'];
-
-      const mergedLabels = Array.from(new Set([...existingLabels, ...severityLabels]));
-
-      updateParams.labels = mergedLabels;
-    }
-
-    const response = await this.octokit.rest.issues.update(updateParams);
-
-    return {
-      number: response.data.number,
-      url: response.data.html_url,
-      labels: response.data.labels.map((l) => (typeof l === 'string' ? l : l.name || ''))
-    };
-  }
-
-  /**
-   * Formats the issue body with dependency metadata
-   */
-  private formatIssueBody(body: string, dependency: { id: string; url: string }): string {
-    return `${body}
-
----
-
-**Dependency Information**
-- ID: \`${dependency.id}\`
-- URL: ${dependency.url}
-
-*This issue was automatically created by dependabit. Add \`false-positive\` or \`true-positive\` label to provide feedback.*`;
-  }
-}