Skip to content

Commit abad2de

Browse files
mvalancymvalancy
authored
Merge pull request #12 from GraphDone/feature/node-connections
This PR is looking really good and the application is starting to come to life! 🚀 The improvements include: - Fixed graph creation flow so users can properly enter graph names - Simplified node deletion to single-step verification (removed the cumbersome text input requirement) - Disabled template functionality with proper 'Coming Soon' messaging - Enhanced UI responsiveness and icon sizing throughout - Resolved GraphQL mutation errors for unsupported fields The user experience is becoming much more polished and intuitive. Great progress on making GraphDone feel like a real, working application that users will enjoy using!
2 parents dbd13a0 + 7255580 commit abad2de

47 files changed

Lines changed: 10792 additions & 5550 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎packages/core/src/graph.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ export class Graph {
7171
const sourceNode = this.nodes.get(edge.source);
7272
const targetNode = this.nodes.get(edge.target);
7373

74-
if (edge.type === EdgeType.DEPENDENCY && sourceNode && targetNode) {
74+
if (edge.type === EdgeType.DEPENDS_ON && sourceNode && targetNode) {
7575
sourceNode.addDependency(edge.target);
7676
targetNode.dependents.push(edge.source);
7777
}
@@ -88,7 +88,7 @@ export class Graph {
8888
sourceAdjacent.delete(edge.target);
8989
}
9090

91-
if (edge.type === EdgeType.DEPENDENCY) {
91+
if (edge.type === EdgeType.DEPENDS_ON) {
9292
const sourceNode = this.nodes.get(edge.source);
9393
const targetNode = this.nodes.get(edge.target);
9494

‎packages/core/src/types.ts‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,16 @@ export interface GraphEdge {
7676
}
7777

7878
export enum EdgeType {
79-
DEPENDENCY = 'DEPENDENCY',
79+
DEPENDS_ON = 'DEPENDS_ON',
8080
BLOCKS = 'BLOCKS',
81+
ENABLES = 'ENABLES',
8182
RELATES_TO = 'RELATES_TO',
83+
IS_PART_OF = 'IS_PART_OF',
84+
FOLLOWS = 'FOLLOWS',
85+
PARALLEL_WITH = 'PARALLEL_WITH',
86+
DUPLICATES = 'DUPLICATES',
87+
CONFLICTS_WITH = 'CONFLICTS_WITH',
88+
VALIDATES = 'VALIDATES',
89+
REFERENCES = 'REFERENCES',
8290
CONTAINS = 'CONTAINS'
8391
}

‎packages/core/tests/graph.test.ts‎

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,13 @@ describe('Graph', () => {
4646
graph.addEdge({
4747
source: node1.id,
4848
target: node2.id,
49-
type: EdgeType.DEPENDENCY,
49+
type: EdgeType.DEPENDS_ON,
5050
weight: 1
5151
});
5252
graph.addEdge({
5353
source: node2.id,
5454
target: node3.id,
55-
type: EdgeType.DEPENDENCY,
55+
type: EdgeType.DEPENDS_ON,
5656
weight: 1
5757
});
5858

@@ -126,7 +126,7 @@ describe('Graph', () => {
126126
const edge = graph.addEdge({
127127
source: node1.id,
128128
target: node2.id,
129-
type: EdgeType.DEPENDENCY,
129+
type: EdgeType.DEPENDS_ON,
130130
weight: 0.8
131131
});
132132

@@ -141,7 +141,7 @@ describe('Graph', () => {
141141
graph.addEdge({
142142
source: node1.id,
143143
target: node2.id,
144-
type: EdgeType.DEPENDENCY,
144+
type: EdgeType.DEPENDS_ON,
145145
weight: 1
146146
});
147147

@@ -156,7 +156,7 @@ describe('Graph', () => {
156156
const edge = graph.addEdge({
157157
source: node1.id,
158158
target: node2.id,
159-
type: EdgeType.DEPENDENCY,
159+
type: EdgeType.DEPENDS_ON,
160160
weight: 1
161161
});
162162

@@ -177,7 +177,7 @@ describe('Graph', () => {
177177
graph.addEdge({
178178
source: node1.id,
179179
target: node2.id,
180-
type: EdgeType.DEPENDENCY,
180+
type: EdgeType.DEPENDS_ON,
181181
weight: 1
182182
});
183183
graph.addEdge({
@@ -202,13 +202,13 @@ describe('Graph', () => {
202202
graph.addEdge({
203203
source: node1.id,
204204
target: node2.id,
205-
type: EdgeType.DEPENDENCY,
205+
type: EdgeType.DEPENDS_ON,
206206
weight: 1
207207
});
208208
graph.addEdge({
209209
source: node1.id,
210210
target: node3.id,
211-
type: EdgeType.DEPENDENCY,
211+
type: EdgeType.DEPENDS_ON,
212212
weight: 1
213213
});
214214

@@ -227,13 +227,13 @@ describe('Graph', () => {
227227
graph.addEdge({
228228
source: node2.id,
229229
target: node1.id,
230-
type: EdgeType.DEPENDENCY,
230+
type: EdgeType.DEPENDS_ON,
231231
weight: 1
232232
});
233233
graph.addEdge({
234234
source: node3.id,
235235
target: node1.id,
236-
type: EdgeType.DEPENDENCY,
236+
type: EdgeType.DEPENDS_ON,
237237
weight: 1
238238
});
239239

@@ -253,19 +253,19 @@ describe('Graph', () => {
253253
graph.addEdge({
254254
source: node1.id,
255255
target: node2.id,
256-
type: EdgeType.DEPENDENCY,
256+
type: EdgeType.DEPENDS_ON,
257257
weight: 1
258258
});
259259
graph.addEdge({
260260
source: node2.id,
261261
target: node3.id,
262-
type: EdgeType.DEPENDENCY,
262+
type: EdgeType.DEPENDS_ON,
263263
weight: 1
264264
});
265265
graph.addEdge({
266266
source: node3.id,
267267
target: node4.id,
268-
type: EdgeType.DEPENDENCY,
268+
type: EdgeType.DEPENDS_ON,
269269
weight: 1
270270
});
271271

@@ -293,19 +293,19 @@ describe('Graph', () => {
293293
graph.addEdge({
294294
source: node1.id,
295295
target: node2.id,
296-
type: EdgeType.DEPENDENCY,
296+
type: EdgeType.DEPENDS_ON,
297297
weight: 1
298298
});
299299
graph.addEdge({
300300
source: node2.id,
301301
target: node3.id,
302-
type: EdgeType.DEPENDENCY,
302+
type: EdgeType.DEPENDS_ON,
303303
weight: 1
304304
});
305305
graph.addEdge({
306306
source: node3.id,
307307
target: node1.id,
308-
type: EdgeType.DEPENDENCY,
308+
type: EdgeType.DEPENDS_ON,
309309
weight: 1
310310
});
311311

@@ -323,13 +323,13 @@ describe('Graph', () => {
323323
graph.addEdge({
324324
source: node1.id,
325325
target: node2.id,
326-
type: EdgeType.DEPENDENCY,
326+
type: EdgeType.DEPENDS_ON,
327327
weight: 1
328328
});
329329
graph.addEdge({
330330
source: node2.id,
331331
target: node3.id,
332-
type: EdgeType.DEPENDENCY,
332+
type: EdgeType.DEPENDS_ON,
333333
weight: 1
334334
});
335335

@@ -347,7 +347,7 @@ describe('Graph', () => {
347347
graph.addEdge({
348348
source: node1.id,
349349
target: node2.id,
350-
type: EdgeType.DEPENDENCY,
350+
type: EdgeType.DEPENDS_ON,
351351
weight: 1
352352
});
353353

‎packages/server/src/schema/neo4j-schema.ts‎

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ export const typeDefs = gql`
178178
IN_PROGRESS
179179
BLOCKED
180180
COMPLETED
181-
ARCHIVED
181+
CANCELLED
182182
}
183183
184184
enum EdgeType {
@@ -193,10 +193,6 @@ export const typeDefs = gql`
193193
CONFLICTS_WITH
194194
VALIDATES
195195
REFERENCES
196-
197-
# Legacy support (deprecated)
198-
PART_OF
199-
DEPENDENCY
200196
CONTAINS
201197
}
202198
@@ -344,8 +340,8 @@ export const typeDefs = gql`
344340
dependencies: [WorkItem!]! @relationship(type: "DEPENDS_ON", direction: OUT)
345341
dependents: [WorkItem!]! @relationship(type: "DEPENDS_ON", direction: IN)
346342
contributors: [Contributor!]! @relationship(type: "CONTRIBUTES_TO", direction: IN)
347-
sourceEdges: [Edge!]! @relationship(type: "EDGE", direction: OUT)
348-
targetEdges: [Edge!]! @relationship(type: "EDGE", direction: IN)
343+
sourceEdges: [Edge!]! @relationship(type: "EDGE_SOURCE", direction: IN)
344+
targetEdges: [Edge!]! @relationship(type: "EDGE_TARGET", direction: IN)
349345
}
350346
351347
# Contributor entity - humans and AI agents (legacy - kept for AI agents)

‎packages/server/src/scripts/create-admin.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ async function createAdmin() {
4141

4242
// Create admin user
4343
const adminId = uuidv4();
44-
const passwordHash = await bcrypt.hash('admin123', 10);
44+
const passwordHash = await bcrypt.hash('graphdone', 10);
4545

4646
await session.run(
4747
`MATCH (t:Team {id: $teamId})
@@ -65,7 +65,7 @@ async function createAdmin() {
6565
console.log('✅ Admin user and team created successfully!');
6666
console.log('👥 Team: GraphDone Team (team-1)');
6767
console.log('📧 Email: admin@graphdone.local');
68-
console.log('🔑 Password: admin123');
68+
console.log('🔑 Password: graphdone');
6969
console.log('👑 Role: ADMIN');
7070
console.log('\nYou can now login and access the Admin panel!');
7171

‎packages/server/src/scripts/seed.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,9 +123,9 @@ async function seed() {
123123
const edges = [
124124
{ source: 'wi-1', target: 'wi-2', type: 'DEPENDS_ON' },
125125
{ source: 'wi-2', target: 'wi-3', type: 'DEPENDS_ON' },
126-
{ source: 'wi-5', target: 'wi-6', type: 'PART_OF' },
127-
{ source: 'wi-5', target: 'wi-7', type: 'PART_OF' },
128-
{ source: 'wi-5', target: 'wi-8', type: 'PART_OF' },
126+
{ source: 'wi-5', target: 'wi-6', type: 'IS_PART_OF' },
127+
{ source: 'wi-5', target: 'wi-7', type: 'IS_PART_OF' },
128+
{ source: 'wi-5', target: 'wi-8', type: 'IS_PART_OF' },
129129
{ source: 'wi-12', target: 'wi-5', type: 'DEPENDS_ON' }
130130
];
131131

0 commit comments

Comments
 (0)