forked from KathiraveluLab/DHGWorkflow
-
Notifications
You must be signed in to change notification settings - Fork 23
Merge the dev branch to the main. #286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
pradeeban
wants to merge
88
commits into
main
Choose a base branch
from
dev
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
840836d
Integrated Monaco Code Editor, Completing Issue #192
saksham-gera c55d9de
Fixing Issue #209, label must contain some text
saksham-gera 2a60f4f
Fixing Issue #208, Unknown Property ‘directory’ in React File Input i…
saksham-gera c76910a
Merge pull request #212 from saksham-gera/issue209
pradeeban f35acd4
Merge pull request #211 from saksham-gera/issue208
pradeeban 91cc8fe
Merge pull request #210 from saksham-gera/issue192
pradeeban 804d55b
Update README.md
pradeeban 0604c8e
fixed workflow failure
saksham-gera a45405e
Merge pull request #215 from saksham-gera/issue192
pradeeban 96d109c
Workflow Issue Fixed
saksham-gera 8e52f16
Alert Issue Fixed #217
saksham-gera 655aef0
Prompt Issue Fixed #218
saksham-gera 0898a3d
Import order fixed
saksham-gera 767865e
Merge pull request #219 from saksham-gera/update_workflow
pradeeban b69af74
Merge pull request #221 from saksham-gera/alert_issue
pradeeban 3b1dc24
Merge pull request #222 from saksham-gera/prompt_issue
pradeeban 15bdf88
Fixed Order Of Imports in FileEdit.jsx
saksham-gera fad247a
Merge pull request #223 from saksham-gera/dev
pradeeban 0b787f1
Merge pull request #224 from ControlCore-Project/dev
pradeeban f80c895
Bump the pip group across 1 directory with 5 updates
dependabot[bot] aa4bb34
Bump the npm_and_yarn group across 1 directory with 14 updates
dependabot[bot] 18473f2
Update buildAndDeploy.yml
saksham-gera 55f77a9
Merge pull request #226 from saksham-gera/dev
pradeeban 626524b
Merge pull request #227 from ControlCore-Project/dev
pradeeban df0215d
Zoom Component Will now update zoom level on zooming through mouse co…
saksham-gera 23eeb04
Merge pull request #229 from saksham-gera/dev
pradeeban 0b4af00
Merge pull request #216 from ControlCore-Project/dependabot/pip/serve…
pradeeban 54a16e1
Added FullScreen Functionality
saksham-gera d260a9c
Merge pull request #231 from saksham-gera/dev
pradeeban 280b0f7
Functionality of loading graphs from previous sessions on hold
saksham-gera 99bddc2
Merge pull request #234 from saksham-gera/dev
pradeeban 53327b1
Added missing dotenv
saksham-gera a89b0f9
Merge pull request #236 from saksham-gera/dev
pradeeban 650699d
Bump the pip group across 1 directory with 5 updates
dependabot[bot] f540d09
Merge branch 'dev' into dependabot/pip/server/pip-3510da4062
pradeeban 164f439
Merge pull request #238 from ControlCore-Project/dependabot/pip/serve…
pradeeban 0bf9eb4
Merge pull request #225 from ControlCore-Project/dependabot/npm_and_y…
pradeeban 56e535e
Merge pull request #239 from ControlCore-Project/dev
pradeeban 3f0f158
Bump the npm_and_yarn group across 1 directory with 5 updates
dependabot[bot] 9e1f348
Bump the pip group across 1 directory with 5 updates
dependabot[bot] 55b4405
Add util
pradeeban 86803c9
Add timers-browserify.
pradeeban caf5063
Attempt to fix deployment
pradeeban f1d4ddb
Merge branch 'dev' into dependabot/pip/server/pip-d578abb6d7
pradeeban e77a3af
Merge pull request #241 from ControlCore-Project/dependabot/pip/serve…
pradeeban b0aec6d
Merge branch 'dev' into dependabot/npm_and_yarn/npm_and_yarn-0d75c3b79c
pradeeban 41af9a7
Merge pull request #240 from ControlCore-Project/dependabot/npm_and_y…
pradeeban 7fe23b3
fix: edges with same prefix(integer) allowed
GREENRAT-K405 780f9f2
Merge pull request #246 from GREENRAT-K405/dev
pradeeban 460e1f6
fix: solve horizontal overflowing problem of text in node
GREENRAT-K405 bb1a501
fix: solve horizontal overflowing problem of text in node
GREENRAT-K405 554690a
Merge pull request #247 from GREENRAT-K405/dev
pradeeban d37a063
fix: Abort error in Open, Save, Save as on pressing cancel button
GREENRAT-K405 5eadbb8
Merge pull request #251 from GREENRAT-K405/dev
pradeeban 3130fad
node aligns with the grids of graph
GREENRAT-K405 4962e05
stricter logic for snapping of nodes with grids
GREENRAT-K405 4e66b26
remove trailing space error
GREENRAT-K405 b0aaf16
remove logs from previous pull req used for testing
GREENRAT-K405 f2ce0ac
remove unimportant logs from fileBrowser.jsx , fileEdit.jsx , 5-load-…
GREENRAT-K405 222a775
Merge pull request #252 from GREENRAT-K405/dev
pradeeban 9bc1dac
Improve the edges: add curve when nodes not aligned, improved visibility
GREENRAT-K405 d7d569f
fix: edges placement when nodes are closed, increased distance betwee…
GREENRAT-K405 8dc73af
Merge pull request #263 from GREENRAT-K405/dev
pradeeban d0c8982
add pop up
GREENRAT-K405 ec5c14a
add pop up to upload directory feature
GREENRAT-K405 e783b34
update close tab pop up logic
GREENRAT-K405 841905b
update upload directory pop-up
GREENRAT-K405 04a7711
update upload directory pop-up
GREENRAT-K405 8936f42
Merge pull request #268 from GREENRAT-K405/dev
pradeeban 6175826
add darkmode to concore editor
GREENRAT-K405 a43387d
complete DARK MODE
GREENRAT-K405 0eb0bf9
update edges dark mode
GREENRAT-K405 7a87a02
update darkmode: removed unnecessary comments
GREENRAT-K405 b215d7c
complete dark mode UI
GREENRAT-K405 7fd3e5f
Merge pull request #269 from GREENRAT-K405/dev
pradeeban f4d8af2
add gemini ai pull request review
GREENRAT-K405 d885d21
Refactor: Replace UA sniffing with feature detection
avinxshKD fe18fd0
Merge branch 'origin/dev' into fix/ua-sniffing
avinxshKD 9292dc7
Fix: Remove incorrect fallback that disabled feature detection
avinxshKD b21bc32
Style: Revert formatting to match upstream
avinxshKD fb62ad6
Cleanup: Remove accidental file and fix type
avinxshKD abb9ba0
add greeings bot
GREENRAT-K405 d007afe
Merge pull request #275 from GREENRAT-K405/dev
pradeeban 59ce547
Merge pull request #276 from avinxshKD/fix/ua-sniffing
pradeeban 35a95d9
update uppercase/lowercase
GREENRAT-K405 febb68d
Merge pull request #279 from GREENRAT-K405/feat/greetings-first-time
pradeeban 8de285b
add missing pr-review.yaml file
GREENRAT-K405 5adc49d
Merge pull request #284 from GREENRAT-K405/dev
pradeeban File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,46 @@ | ||
| name: AI Code Reviewer | ||
|
|
||
| on: | ||
| issue_comment: | ||
| types: [created] | ||
|
|
||
| permissions: | ||
| contents: read | ||
| pull-requests: write | ||
| issues: write | ||
|
|
||
| jobs: | ||
| gemini-code-review: | ||
| runs-on: ubuntu-latest | ||
| if: | | ||
| github.event.issue.pull_request && | ||
| contains(github.event.comment.body, '/gemini-review') | ||
| steps: | ||
| - name: PR Info | ||
| run: | | ||
| echo "Comment: ${{ github.event.comment.body }}" | ||
| echo "Issue Number: ${{ github.event.issue.number }}" | ||
| echo "Repository: ${{ github.repository }}" | ||
|
|
||
| - name: Checkout Repo | ||
| uses: actions/checkout@v3 | ||
| with: | ||
| fetch-depth: 0 | ||
| ref: refs/pull/${{ github.event.issue.number }}/head | ||
|
|
||
| - name: Get PR Details | ||
|
Comment on lines
+25
to
+31
Check failureCode scanning / CodeQL Checkout of untrusted code in trusted context High
Potential execution of untrusted code on a privileged workflow (
issue_comment Error loading related location Loading |
||
| id: pr | ||
| run: | | ||
| PR_JSON=$(gh api repos/${{ github.repository }}/pulls/${{ github.event.issue.number }}) | ||
| echo "head_sha=$(echo $PR_JSON | jq -r .head.sha)" >> $GITHUB_OUTPUT | ||
| echo "base_sha=$(echo $PR_JSON | jq -r .base.sha)" >> $GITHUB_OUTPUT | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - uses: truongnh1992/gemini-ai-code-reviewer@main | ||
| with: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| GEMINI_API_KEY: ${{ secrets.GEMINI_API_KEY }} | ||
| GEMINI_MODEL: gemini-2.5-flash | ||
| EXCLUDE: "*.md,*.txt,package-lock.json" | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,26 +1,33 @@ | ||
| name: Build and Deploy | ||
|
|
||
| on: workflow_dispatch | ||
|
|
||
| jobs: | ||
| build-and-deploy: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| node-version: [16.x] | ||
| node-version: [18.x] | ||
| steps: | ||
| - name: Checkout 🛎️ | ||
| uses: actions/checkout@v2.3.1 | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Use Node.js ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v3 | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
|
|
||
| - name: Clear npm cache | ||
| run: npm cache clean --force | ||
|
|
||
| - name: Install and Build 🔧 | ||
| run: | | ||
| export NODE_OPTIONS=--openssl-legacy-provider | ||
| npm ci | ||
| npm run build | ||
|
|
||
| - name: Deploy 🚀 | ||
| uses: JamesIves/github-pages-deploy-action@4.1.3 | ||
| uses: JamesIves/github-pages-deploy-action@v4 | ||
| with: | ||
| branch: gh-pages # The branch the action should deploy to. | ||
| folder: build # The folder the action should deploy. | ||
| branch: gh-pages | ||
| folder: build |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,41 @@ | ||
| name: Greetings | ||
|
|
||
| on: | ||
| pull_request_target: | ||
| types: [opened] | ||
| issues: | ||
| types: [opened] | ||
|
|
||
| jobs: | ||
| greeting: | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| issues: write | ||
| pull-requests: write | ||
| steps: | ||
| - uses: actions/first-interaction@v3 | ||
| with: | ||
| repo_token: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| issue_message: | | ||
| 👋 Welcome to the CONTROL-CORE Project, @${{ github.actor }}! Thank you for opening your first issue in concore-editor. | ||
| We appreciate your contribution to the organization and will review it as soon as possible. | ||
|
|
||
| Before we get started, please check out these resources: | ||
| - 📚 [Project Documentation](https://control-core.readthedocs.io/) | ||
| - 📘 [Contribution Guidelines](https://github.com/ControlCore-Project/concore/blob/main/CONTRIBUTING.md) | ||
| - 📜 [Code of Conduct](https://github.com/ControlCore-Project/concore/blob/main/CODE_OF_CONDUCT.md) | ||
|
|
||
| pr_message: | | ||
| 🎉 Welcome aboard, @${{ github.actor }}! Thank you for your first pull request in concore-editor. | ||
|
|
||
| Please ensure that you are contributing to the **dev** branch. | ||
|
|
||
| Your contribution means a lot to us. We'll review it shortly. | ||
|
|
||
| Please ensure you have reviewed our: | ||
| - 📚 [Project Documentation](https://control-core.readthedocs.io/) | ||
| - 📘 [Contribution Guidelines](https://github.com/ControlCore-Project/concore/blob/main/CONTRIBUTING.md) | ||
| - 📜 [Code of Conduct](https://github.com/ControlCore-Project/concore/blob/main/CODE_OF_CONDUCT.md) | ||
|
|
||
| If you have any questions, feel free to ask. Happy coding! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| const webpack = require('webpack'); | ||
|
|
||
| module.exports = { | ||
| webpack: { | ||
| configure: (webpackConfig) => { | ||
| const newConfig = { ...webpackConfig }; // copy the original config | ||
|
|
||
| newConfig.resolve = { | ||
| ...newConfig.resolve, | ||
| fallback: { | ||
| ...newConfig.resolve.fallback, | ||
| timers: require.resolve('timers-browserify'), | ||
| util: require.resolve('util/'), | ||
| stream: require.resolve('stream-browserify'), | ||
| buffer: require.resolve('buffer/'), | ||
| process: require.resolve('process/browser.js'), | ||
| }, | ||
| }; | ||
|
|
||
| newConfig.plugins = [ | ||
| ...newConfig.plugins, | ||
| new webpack.ProvidePlugin({ | ||
| Buffer: ['buffer', 'Buffer'], | ||
| process: 'process/browser.js', | ||
| }), | ||
| ]; | ||
|
|
||
| return newConfig; | ||
| }, | ||
| }, | ||
| }; |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check failure
Code scanning / CodeQL
Code injection Critical
Copilot Autofix
AI about 24 hours ago
General fix: Avoid using
${{ github.event.comment.body }}directly in arunstep. Instead, assign it to an environment variable in the step’senv:block using expression syntax, and reference that variable using shell syntax ($VAR) inside the script.Concrete best fix here:
PR Infostep (lines 19–24), movegithub.event.comment.body,github.event.issue.number, andgithub.repositoryinto environment variables via anenv:block.runscript to use$COMMENT_BODY,$ISSUE_NUMBER, and$REPOSITORYinstead of${{ ... }}.Changes are limited to the
PR Infostep in.github/workflows/PR-review.yamlaround lines 19–24. No new imports or external dependencies are needed.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GREENRAT-K405 see if this solution makes sense (and if it does, please use that in your fix.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pradeeban I have made changes in PR-review.yaml and will raise PR soon in all three repos.