Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 11, 2025

Removes the AI-powered GitHub issue generation feature per requirements.

Changes

Routes removed (blt/urls.py):

  • /github-issue-prompt/ - Issue generation form
  • /create-github-issue/ - GitHub issue creation endpoint
  • /get-github-issue/ - AI generation processor

Views removed (website/views/issue.py):

  • GithubIssueView - Issue creation handler (44 lines)
  • get_github_issue() - Generation request processor (27 lines)
  • generate_github_issue() - OpenAI API integration (45 lines)
  • OpenAI import (still used in utils.py and bot.py)

Templates deleted:

  • github_issue_prompt.html - Input form (94 lines)
  • github_issue.html - Issue creation form (246 lines)

UI cleanup:

  • Navigation links removed from header and sidenav
  • Feature card removed from features page
  • Static asset generate-issue.png deleted
  • Duplicate keywords cleaned from features page metadata

Total: 510 lines removed across 8 files

Original prompt

Remove the generate issue feature


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 11, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI and others added 2 commits December 11, 2025 07:40
- Remove URL routes for github-issue-prompt, create-github-issue, and get-github-issue
- Remove GithubIssueView class and generate_github_issue function from views
- Remove github_issue_prompt.html and github_issue.html templates
- Remove navigation links from header and sidenav
- Remove feature card from features.html
- Remove generate-issue.png static image
- Remove OpenAI import from issue.py (still used in other files)

Co-authored-by: DonnieBLT <128622481+DonnieBLT@users.noreply.github.com>
Co-authored-by: DonnieBLT <128622481+DonnieBLT@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove the generate issue feature Remove GitHub issue generator feature Dec 11, 2025
Copilot AI requested a review from DonnieBLT December 11, 2025 07:46
@github-actions github-actions bot added the files-changed: 8 PR changes 8 files label Dec 11, 2025
@DonnieBLT DonnieBLT requested a review from Copilot December 11, 2025 07:47
@github-actions
Copy link
Contributor

❌ Pre-commit checks failed

The pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them:

# Install pre-commit if you haven't already
pip install pre-commit

# Run pre-commit on all files
pre-commit run --all-files

# Or run pre-commit on staged files only
pre-commit run

After running these commands, the pre-commit hooks will automatically fix most issues.
Please review the changes, commit them, and push to your branch.

💡 Tip: You can set up pre-commit to run automatically on every commit by running:

pre-commit install
Pre-commit output
[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/pycqa/isort.
[WARNING] repo `https://github.com/pycqa/isort` uses deprecated stage names (commit, merge-commit, push) which will be removed in a future version.  Hint: often `pre-commit autoupdate --repo https://github.com/pycqa/isort` will fix this.  if it does not -- consider reporting an issue to that repo.
[INFO] Initializing environment for https://github.com/astral-sh/ruff-pre-commit.
[INFO] Initializing environment for https://github.com/djlint/djLint.
[INFO] Initializing environment for local.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/pycqa/isort.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/astral-sh/ruff-pre-commit.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/djlint/djLint.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for local.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
check python ast.........................................................Passed
check builtin type constructor use.......................................Passed
check yaml...............................................................Passed
fix python encoding pragma...............................................Passed
mixed line ending........................................................Passed
isort....................................................................Passed
ruff.....................................................................Failed
- hook id: ruff
- files were modified by this hook

Found 2 errors (2 fixed, 0 remaining).


For more information, see the pre-commit documentation.

@github-actions github-actions bot added the pre-commit: failed Pre-commit checks failed label Dec 11, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR completely removes the AI-powered GitHub issue generation feature from BLT. The removal is comprehensive, eliminating all routes, views, templates, navigation links, and static assets related to the feature. This cleanup removes 510 lines of code including form templates that contained style tag violations of the project's coding guidelines.

Key changes:

  • Removed AI-powered issue generation routes and views (116 lines of Python code)
  • Deleted two HTML templates totaling 340 lines
  • Cleaned up navigation links and feature page entries, including duplicate keywords

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
blt/urls.py Removed 3 URL patterns (github-issue-prompt, create-github-issue, get-github-issue) and corresponding imports for GithubIssueView and get_github_issue
website/views/issue.py Removed GithubIssueView class (44 lines), get_github_issue() function (27 lines), generate_github_issue() function (45 lines), and OpenAI import (still used elsewhere in utils.py)
website/templates/github_issue_prompt.html Deleted entire template file (94 lines) - the input form for AI generation
website/templates/github_issue.html Deleted entire template file (246 lines) - the issue creation form with style tag violations
website/templates/includes/sidenav.html Removed navigation link to github_issue_prompt from sidebar
website/templates/includes/header.html Removed navigation link to github_issue_prompt from header menu
website/templates/features.html Removed AI-Powered GitHub Issue Generator feature card and cleaned up duplicate keywords (Scoreboard, Bug Bounties, Leaderboard, etc.)
website/static/img/features/generate-issue.png Deleted static asset image file

@github-actions github-actions bot added tests: passed Django tests passed last-active: 0d PR last updated 0 days ago labels Dec 11, 2025
@github-actions github-actions bot added last-active: 2d PR last updated 2 days ago and removed last-active: 0d PR last updated 0 days ago labels Dec 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

files-changed: 8 PR changes 8 files last-active: 2d PR last updated 2 days ago pre-commit: failed Pre-commit checks failed tests: passed Django tests passed

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants